Skip to content

Conversation

@dgw
Copy link
Member

@dgw dgw commented Feb 2, 2025

Trying out a new workflow: I've merged the changelog and backport branches for 8.0.2 into this PR, which will both run CI on the final result and allow other maintainers to give the overall patch a once-over before it gets merged for final release.

If we like doing things this way, we can use it for 8.1.0—and create (an) appropriate label(s) in the tracker for this type of PR.

dgw added 9 commits January 25, 2025 18:51
build: pin urllib3<2.3 to accommodate our old fork of vcrpy

Also includes slight `typing.cast()` changes to accommodate new version
of flake8-type-checking.
privileges, url: fix privilege check in `.urlban` command family
safety: fix safeify_url() exception on python 3.11
bot: round rate limit `time_left` value up, eliminate microseconds
@dgw dgw added this to the 8.0.2 milestone Feb 2, 2025
@dgw dgw requested review from Exirel and SnoopJ February 2, 2025 21:19
@dgw
Copy link
Member Author

dgw commented Feb 2, 2025

which will both run CI on the final result

Well it's supposed to do that. I will attempt to tweak this config and make the branch name v prefix optional:

pull_request:
branches:
- master
- v[0-9]+.[0-9]+.x

@dgw
Copy link
Member Author

dgw commented Feb 2, 2025

OK, the CI configuration is now working as expected for 8.0.x: It runs on PRs, and includes Python 3.13 in the version matrix.

@dgw
Copy link
Member Author

dgw commented Feb 5, 2025

17:21:50 <+Sopel> [sopel] dgw commented on pull request #2655: I figure there's no need to go through a whole
new PR just to pull an added `dev-requirements.txt` comment line from #2654 upstream into `master`. I will
get @SnoopJ's sign-off on the new comment via IRC (it was his idea).
https://github.com/sopel-irc/sopel/pull/2655#issuecomment-2638257122
17:22:57 <+SnoopJ> :+1: 
17:24:08 <+dgw> goes for 2654 too I hope?
17:24:12 <+SnoopJ> ye

@dgw dgw merged commit a794b75 into 8.0.x Feb 5, 2025
13 checks passed
@dgw dgw deleted the 8.0.2-prep branch February 5, 2025 23:25
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.

4 participants