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

add PR number to and touch up NEWS.md entry for #18754 #21842

Merged
merged 1 commit into from
May 24, 2017

Conversation

Sacha0
Copy link
Member

@Sacha0 Sacha0 commented May 13, 2017

Ref. #21475. Best!

@Sacha0 Sacha0 added the needs news A NEWS entry is required for this change label May 13, 2017
@Sacha0 Sacha0 added this to the 0.6.0 milestone May 13, 2017

* All Intrinsics are now Builtin functions instead and have proper error checking
and fall-back static compilation support.
* All `Intrinsics` are now `Builtin` functions instead and have proper error checking
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think these have code meanings, should maybe just be not capitalized

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed quoting and capitalization. Thanks!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh they exist in Core, but are very sparsely documented in the devdocs

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quoting and capitalization restored. How now? :)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pretty sure the module name is plural

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed. Thanks!

NEWS.md Outdated
@@ -584,3 +585,4 @@ Deprecated or removed
[#20543]: https://github.com/JuliaLang/julia/issues/20543
[#20609]: https://github.com/JuliaLang/julia/issues/20609
[#20889]: https://github.com/JuliaLang/julia/issues/20889
[#21359]: https://github.com/JuliaLang/julia/issues/21359
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this part of the diff shouldn't be backported

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch! I stripped this line from the diff here and elsewhere. I will strip this line from any future commits as well. Thanks!

@Sacha0 Sacha0 changed the title add PR number and quotes to NEWS.md entry for #18754 add PR number to and touch up NEWS.md entry for #18754 May 13, 2017
@tkelman tkelman merged commit e020772 into JuliaLang:master May 24, 2017
@Sacha0 Sacha0 deleted the news18754 branch May 25, 2017 17:44
@Sacha0 Sacha0 removed the needs news A NEWS entry is required for this change label May 25, 2017
tkelman pushed a commit that referenced this pull request Jun 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants