Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Duplicated mapping key error should point to the key #243

Closed
dervus opened this issue Jan 11, 2016 · 11 comments
Closed

Duplicated mapping key error should point to the key #243

dervus opened this issue Jan 11, 2016 · 11 comments
Labels
bug improvement

Comments

@dervus
Copy link
Collaborator

dervus commented Jan 11, 2016

As revealed in #242 the error points to end of key-value pair instead of duplicate key itself, which is confusing.
@dervus dervus mentioned this issue Feb 15, 2016
Closed
@KSDaemon
Copy link

+1 Really sad :(

@mysticatea mysticatea mentioned this issue Mar 15, 2016
Closed
@dervus dervus mentioned this issue May 10, 2016
Closed
@stefan-dimitrov stefan-dimitrov mentioned this issue Jul 11, 2016
Closed
@luengnat
Copy link

luengnat commented Aug 4, 2016

@dervus how can we fix this issue?

@puzrin
Copy link
Member

puzrin commented Aug 4, 2016

@luengnat via PR, if you can't wait anymore :)

@syzer
Copy link

syzer commented Aug 22, 2016

+1

2 similar comments
@tea3
Copy link

tea3 commented Sep 2, 2016

+1

@filipnowakswissre
Copy link

+1

@highfeed
Copy link

+3

@iDevy
Copy link

iDevy commented Oct 20, 2016

+1

1 similar comment
@andrewdollard
Copy link

+1

@stefan-dimitrov stefan-dimitrov mentioned this issue Nov 15, 2016
Closed
@sputnick
Copy link

+1

@puzrin puzrin added the bug label Nov 26, 2016
This was referenced Nov 27, 2016
Open
Open
Open
@taot
Copy link

taot commented Jan 7, 2017

+1

@shockey shockey mentioned this issue Feb 3, 2017
Merged
imbstack added a commit to taskcluster/taskcluster-github that referenced this issue Feb 21, 2017
@imbstack
Upgrade js-yml to get their fix for nodeca/js-yaml#243
7c55062
@imbstack imbstack mentioned this issue Feb 21, 2017
Merged
owlishDeveloper pushed a commit to taskcluster/taskcluster-github that referenced this issue Feb 22, 2017
@imbstack @owlishDeveloper
Upgrade js-yml to get their fix for nodeca/js-yaml#243
04b0be8
@petemoore petemoore mentioned this issue Feb 22, 2017
Closed
@hughsw hughsw mentioned this issue Oct 22, 2018
Closed
rlidwka added a commit that referenced this issue Dec 11, 2020
@rlidwka
Remember cursor position for 'duplicate mapping key' exception
b1cc0e9 
ref #452
ref #243
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug improvement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests