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.

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

give a syntax error for repeated keyword args. fixes #16937 #21780

Merged
merged 1 commit into from
May 17, 2017

Conversation

JeffBezanson
Copy link
Sponsor Member

@JeffBezanson JeffBezanson commented May 10, 2017

I'm ok with just adding this error without a deprecation. This should be very uncommon and never did anything useful.

@JeffBezanson JeffBezanson added kind:breaking This change will break code domain:error handling Handling of exceptions by Julia or the user compiler:lowering Syntax lowering (compiler front end, 2nd stage) labels May 10, 2017
@tkelman
Copy link
Contributor

tkelman commented May 10, 2017

should go in news

@tkelman
Copy link
Contributor

tkelman commented May 10, 2017

can we avoid moving the file to history until we've tagged final? especially given the remaining item on the milestone

@JeffBezanson
Copy link
Sponsor Member Author

Ok I'll drop the second commit, wait until 0.6-final, and then put in a news item here.

@tkelman
Copy link
Contributor

tkelman commented May 10, 2017

starting a separate 0.7 section of the file without moving it yet would also not conflict with any backports. a few open pr's have done so but guess none have been merged yet

@JeffBezanson JeffBezanson added the needs news A NEWS entry is required for this change label May 10, 2017
@JeffBezanson JeffBezanson removed the needs news A NEWS entry is required for this change label May 16, 2017
@JeffBezanson JeffBezanson merged commit 4a99ce4 into master May 17, 2017
@JeffBezanson JeffBezanson deleted the jb/fix16937 branch May 17, 2017 03:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compiler:lowering Syntax lowering (compiler front end, 2nd stage) domain:error handling Handling of exceptions by Julia or the user kind:breaking This change will break code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants