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

Fix issue 20527. Set the correct union member for integers so CTFE #7361

Merged
merged 1 commit into from
Jan 23, 2020
Merged

Fix issue 20527. Set the correct union member for integers so CTFE #7361

merged 1 commit into from
Jan 23, 2020

Conversation

schveiguy
Copy link
Member

doesn't get confused.

@dlang-bot
Copy link
Contributor

Thanks for your pull request, @schveiguy!

Bugzilla references

Auto-close Bugzilla Severity Description
✓ 20527 normal std.json CTFE cannot read integer

Testing this PR locally

If you don't have a local development environment setup, you can use Digger to test this PR:

dub fetch digger
dub run digger -- build "stable + phobos#7361"

@schveiguy
Copy link
Member Author

I can't figure out what buildkite is doing or how to knock it loose.

@schveiguy schveiguy merged commit 5e13653 into dlang:stable Jan 23, 2020
@schveiguy schveiguy deleted the fixjsonctfe branch January 24, 2020 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@PetarKirov PetarKirov PetarKirov approved these changes

@CyberShadow CyberShadow CyberShadow approved these changes

Assignees
No one assigned
Labels
Merge:auto-merge Severity:Bug Fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@schveiguy @dlang-bot @CyberShadow @PetarKirov