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

CompatHelper: bump compat for "Reexport" to "1.0" #2587

Merged

Conversation

github-actions[bot]
Copy link
Contributor

This pull request changes the compat entry for the Reexport package from 0.1, 0.2 to 0.1, 0.2, 1.0.

This keeps the compat entries for earlier versions.

Note: I have not tested your package with this new compat entry. It is your responsibility to make sure that your package tests pass before you merge this pull request.

@bkamins
Copy link
Member

bkamins commented Dec 26, 2020

@ararslan - I understand it is safe to merge it once https://github.com/simonster/Reexport.jl is tagged. Right?

@bkamins bkamins added this to the 1.0 milestone Dec 26, 2020
@bkamins bkamins added backport ecosystem Issues in DataFrames.jl ecosystem non-breaking The proposed change is not breaking labels Dec 26, 2020
@nalimilan
Copy link
Member

Any idea why CI doesn't run?

@bkamins
Copy link
Member

bkamins commented Dec 28, 2020

I do not know - it happened sometimes in the past also, but I do not know the rule here.

@nalimilan
Copy link
Member

Closing and reopening works. Unfortunately, CI used 0.2.0 because of dependencies, so we shouldn't merge this for now.

Copy link
Member

@ararslan ararslan left a comment

Choose a reason for hiding this comment

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

IMO it's fine to merge as-is since Reexport 1.0 doesn't actually have any breaking changes compared to 0.2, so I wouldn't expect anything to be noticeably affected for existing users.

@ararslan
Copy link
Member

ararslan commented Jan 4, 2021

@ararslan - I understand it is safe to merge it once https://github.com/simonster/Reexport.jl is tagged. Right?

Pkg doesn't care about git tags, so as long as 1.0 is in the registry (which it seems to be given that this PR was opened) then it's safe.

@ararslan
Copy link
Member

ararslan commented Jan 4, 2021

(I just made a tag anyway)

@bkamins bkamins merged commit 4804525 into main Jan 4, 2021
@bkamins bkamins deleted the compathelper/new_version/2020-12-26-00-36-20-165-2050621716 branch January 4, 2021 19:51
@bkamins
Copy link
Member

bkamins commented Jan 4, 2021

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport ecosystem Issues in DataFrames.jl ecosystem non-breaking The proposed change is not breaking
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants