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

Bump patch #849

Closed
wants to merge 4 commits into from
Closed

Bump patch #849

wants to merge 4 commits into from

Conversation

willtebbutt
Copy link
Member

Looks like a patch was skipped when #844 was merged.

@DhairyaLGandhi
Copy link
Member

Is this meant to tag a breeding release

@willtebbutt
Copy link
Member Author

willtebbutt commented Dec 8, 2020

Hmmm no. It appears that the Project.toml was just bumped to 0.6.0. Not sure when / how that happened as I can't see the corresponding PR.

@willtebbutt
Copy link
Member Author

Oh, looks like I just missed this being merged in.

@CarloLucibello
Copy link
Member

we didn't even tag v0.6 yet, and we have a few PRs to merge before that https://github.com/FluxML/Zygote.jl/milestone/2

@willtebbutt
Copy link
Member Author

we didn't even tag v0.6 yet, and we have a few PRs to merge before that https://github.com/FluxML/Zygote.jl/milestone/2

Yup, I've now seen that. Could you maybe branch this stuff out into a 0.6-DEV branch or something so that non-breaking changes to 0.5 aren't blocked?

@willtebbutt
Copy link
Member Author

Closing for now. Looks like my merged code will get included in 0.6, which is fine.

@willtebbutt willtebbutt closed this Dec 9, 2020
@willtebbutt willtebbutt deleted the wct/bump-patch branch January 16, 2021 12:44
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.

3 participants