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

update for newer General branch format #264

Merged

Conversation

IanButterworth
Copy link
Member

@IanButterworth IanButterworth commented Jul 24, 2023

Fixes #263
Fixes #258
Fixes #262

Not sure how to test this before merge given tests didn't fail while this was broken.

@DilumAluthge DilumAluthge requested review from christopher-dG and removed request for christopher-dG July 24, 2023 03:17
@IanButterworth IanButterworth changed the title update for newer Genreral branch format update for newer General branch format Jul 24, 2023
@IanButterworth IanButterworth force-pushed the ib/general_branch_fix branch 3 times, most recently from e0f0acd to e8d1ef7 Compare July 24, 2023 04:12
@IanButterworth
Copy link
Member Author

@christopher-dG are you the right person to ping on this? If not who'd be good?

@IanButterworth
Copy link
Member Author

Bump

tagbot/action/repo.py Outdated Show resolved Hide resolved
tagbot/action/repo.py Outdated Show resolved Hide resolved
@IanButterworth IanButterworth force-pushed the ib/general_branch_fix branch 10 times, most recently from 764ca4f to 3507e83 Compare August 23, 2023 03:19
Copy link
Member

@christopher-dG christopher-dG left a comment

Choose a reason for hiding this comment

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

Minor nitpicks, looks good 👍

tagbot/action/repo.py Outdated Show resolved Hide resolved
tagbot/action/repo.py Outdated Show resolved Hide resolved
Copy link
Member

@christopher-dG christopher-dG left a comment

Choose a reason for hiding this comment

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

Great! Can you bump the patch version in pyproject.toml?

@christopher-dG christopher-dG merged commit 6dc4354 into JuliaRegistries:master Aug 25, 2023
3 checks passed
@IanButterworth
Copy link
Member Author

IanButterworth commented Aug 25, 2023

It'd be good to test this in the wild. Does the bot pick up new releases immediately?

@IanButterworth IanButterworth deleted the ib/general_branch_fix branch August 25, 2023 13:12
@christopher-dG
Copy link
Member

I actually realized that I should not have asked you to bump the version yourself, since the release script does that automatically. Oops! I'll revert just that change then run the release workflow.

@IanButterworth
Copy link
Member Author

Sounds good. Worth getting any other dep bump PRs in while we're here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants