Skip to content

Tools for updating contributors and version numbers #2734

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
Merged
merged 40 commits into from
Sep 12, 2024
Merged

Tools for updating contributors and version numbers #2734

merged 40 commits into from
Sep 12, 2024

Conversation

@tomc271
Copy link
Collaborator

@tomc271 tomc271 commented Jul 11, 2023

No description provided.

@tomc271 tomc271 requested a review from ZedThree July 11, 2023 10:08
dschwoerer
dschwoerer requested changes Jul 12, 2023
View reviewed changes
Copy link
Contributor

@dschwoerer dschwoerer left a comment

Choose a reason for hiding this comment

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

I was confused why this was not black formatted, but I think that's because it is in a fork, and CI is not enabled.

@tomc271 tomc271 marked this pull request as draft July 12, 2023 10:16
@tomc271 tomc271 force-pushed the automate-release-process branch from a5e7f09 to b4db274 Compare July 13, 2023 16:30
ZedThree
ZedThree reviewed Aug 11, 2023
View reviewed changes
@tomc271 tomc271 force-pushed the automate-release-process branch from 47342bf to 80eced1 Compare August 21, 2023 17:15
@tomc271 tomc271 marked this pull request as ready for review August 21, 2023 18:11
@ZedThree ZedThree changed the base branch from next to v5.1.0-rc September 20, 2023 13:17
ZedThree
ZedThree previously approved these changes Sep 20, 2023
View reviewed changes
dschwoerer
dschwoerer requested changes Sep 20, 2023
View reviewed changes
@ZedThree ZedThree force-pushed the automate-release-process branch from 33632ad to 319fb11 Compare September 12, 2024 12:54
@ZedThree ZedThree changed the base branch from v5.1.0-rc to v5.1.1-rc September 12, 2024 12:59
@tomchapman @ZedThree
Add script to bump version numbers. (Initial revision).
2f797bc 
[skip ci]
@tomchapman @ZedThree
Add script to check for new authors. (Initial revision).
3a5bda2 
[skip ci]
@tomchapman @ZedThree
Move scripts to bin directory
ab8deca
@tomchapman @ZedThree
Remove hard-coded paths.
e397570
@tomchapman @ZedThree
Make KNOWN_AUTHORS a dictionary, holding more information to help ide…
039f117 
…ntify the known authors.
@tomchapman @ZedThree
Handle version numbers with multiple digits.
8d0e0f8
@tomchapman @ZedThree
Add classes VersionNumber and ShortVersionNumber.
a248468
@tomchapman @ZedThree
Run git log command to get the list of authors.
9a502e1
@tomchapman @ZedThree
Extracted function get_main_directory().
bec2572
@tomchapman @ZedThree
Also list emails for unrecognised authors.
bb6f4e1 
[skip ci]
@tomchapman @ZedThree
Copy functions from other scripts:
1058454 
apply_fixes(replacements, source)
yes_or_no(question)
create_patch(filename, original, modified)
[unfinished]
@tomchapman @ZedThree
Apply copied functions to this script.
120af43
@tomchapman @ZedThree
Black formatting
1c28f31 
[skip ci]
@tomchapman @ZedThree
No need for ShortVersionNumber to inherit from VersionNumber
6c91918
@tomchapman @ZedThree
Use cwd argument to subprocess.run() to change directory (subprocess.…
170b1dc 
…run("cd",... does nothing as it's a separate process).

[skip ci]
@tomchapman @ZedThree
Use subprocess check=True argument to check for error.
6a8897d 
[skip ci]
tomchapman and others added 24 commits September 12, 2024 14:16
@tomchapman @ZedThree
Use subprocess text=True argument (then no need to decode() stdout).
5d68609 
[skip ci]
@tomchapman @ZedThree
No need to catch real errors.
0278489 
[skip ci]
@tomchapman @ZedThree
When catching KeyError, raise a new exception instead. Use an f-string.
72c0637 
[skip ci]
@tomchapman @ZedThree
Use get_main_directory() function.
ae072d8 
[skip ci]
@tomchapman @ZedThree
Make VersionNumber a dataclass.
d10dbe8 
[skip ci]
@tomchapman @ZedThree
Use str dunder method.
65d42bc 
[skip ci]
@tomchapman @ZedThree
Remove commented-out copied code.
9570f9f 
[skip ci]
@tomchapman @ZedThree
Add command line argument for the new version number.
6e5f6fa 
[skip ci]
@tomchapman @ZedThree
Corrected regex.
871d946 
[skip ci]
@tomchapman @ZedThree
Add command line description to update_version_number_in_file script.
e779059 
[skip ci]
@tomchapman @ZedThree
Make ShortVersionNumber a dataclass.
d031412 
[skip ci]
@ZedThree
Maint: Make update version/citation scripts executable
8fe1dfe 
[skip ci]
@ZedThree
Maint: Fix typo in next version regex
a8ae191
@ZedThree
Maint: Update list of known authors in citation script
0ef56ad
@ZedThree
Maint: Don't print anything if no unrecognised authors
4a02f4c
@ZedThree @dschwoerer
Fix typo in author list
8a3b237 
[skip ci]

Co-authored-by: David Bold <dschwoerer@users.noreply.github.com>
@dschwoerer @ZedThree
Fix author alias in citation update tool
356bf9b
@ZedThree
Add list of packages for maintenance tasks
d864924
@ZedThree
Fix some formatting in citation updater
240965a
@ZedThree
Make citation updater patch citation file
37bb0e8
@ZedThree
Remove extra "v" from previous version regexp
6a1a997
@ZedThree
Replace short version class with property
86a10a1
@ZedThree
Add optional next version number argument to version updater script
ebdfd7d
@ZedThree
Make some regexes a little more robust
15e2e7e
@ZedThree ZedThree force-pushed the automate-release-process branch from 319fb11 to 15e2e7e Compare September 12, 2024 13:16
@ZedThree ZedThree changed the title Automate release process Tools for updating contributors and version numbers Sep 12, 2024
@ZedThree ZedThree merged commit b7c3534 into boutproject:v5.1.1-rc Sep 12, 2024
10 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dschwoerer dschwoerer dschwoerer requested changes

@ZedThree ZedThree ZedThree approved these changes

Assignees

No one assigned

Labels

release

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@tomc271 @ZedThree @dschwoerer @tomchapman