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

ci: Migrate to commitizen to bump version #348

Merged
merged 4 commits into from
Sep 27, 2023

Conversation

OliverCullimore
Copy link
Collaborator

@OliverCullimore OliverCullimore commented Sep 27, 2023

This should hopefully fix the automated releases.

@robbrad The bump environment will require a Personal Access Token to be passed in via the "PERSONAL_ACCESS_TOKEN" secret with the following configuration if you can set that up before this one's merged:

  • Limited to this repository using "Only select repositories"
  • "Read and Write" access to "Contents"
  • "Read-only" access to "Metadata"

This is to enable the action to update the pyproject.toml and manifest.json files with the new version number without creating a second PR we'd have to accept after each merge.

I've tested this in a copy of the repo so hopefully it'll all just work first time and we can close #329 #346 #347.

Update to also address #326 by using the corresponding tag's version of the file.

@OliverCullimore OliverCullimore added bug Something isn't working github_actions Pull requests that update GitHub Actions code labels Sep 27, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #348 (2edac7d) into master (e9c151a) will not change coverage.
The diff coverage is n/a.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@           Coverage Diff           @@
##           master     #348   +/-   ##
=======================================
  Coverage   83.44%   83.44%           
=======================================
  Files           3        3           
  Lines         151      151           
=======================================
  Hits          126      126           
  Misses         25       25           

@robbrad
Copy link
Owner

robbrad commented Sep 27, 2023

I think I did it right !

@OliverCullimore OliverCullimore merged commit a99b311 into robbrad:master Sep 27, 2023
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working github_actions Pull requests that update GitHub Actions code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bolton Council Broken
3 participants