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

Add PAT to automatically commit release notes to CHANGELOG.md #695

Merged
merged 5 commits into from
Jun 8, 2022

Conversation

JulioAPeraza
Copy link
Collaborator

Closes None.

Changes proposed in this pull request:

  • Add PAT from my account to automatically commit release notes to CHANGELOG.

Note: In the future, it would be great to create an organizational bot account for these tasks.

@JulioAPeraza JulioAPeraza added the documentation Improving documentation for the project label Jun 2, 2022
@codecov
Copy link

codecov bot commented Jun 2, 2022

Codecov Report

Merging #695 (046590e) into main (e0d7cf5) will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #695      +/-   ##
==========================================
+ Coverage   85.28%   85.31%   +0.02%     
==========================================
  Files          41       41              
  Lines        4507     4521      +14     
==========================================
+ Hits         3844     3857      +13     
- Misses        663      664       +1     
Impacted Files Coverage Δ
nimare/utils.py 94.52% <0.00%> (-0.17%) ⬇️
nimare/meta/cbma/mkda.py 97.08% <0.00%> (-0.01%) ⬇️
nimare/meta/utils.py 54.29% <0.00%> (+1.49%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e0d7cf5...046590e. Read the comment docs.

@JulioAPeraza
Copy link
Collaborator Author

I don't know how to test this on a PR. I think we will need to merge this again and make another pre-release.

@tsalo
Copy link
Member

tsalo commented Jun 8, 2022

I can merge this now if you want to test it out on a new release. Is it ready?

@JulioAPeraza
Copy link
Collaborator Author

Yes, I think this is ready. If it doesn't work this time, I will try creating a bot account to push the Update CHANGELOG.md commits.

@tsalo tsalo merged commit 95f9bce into neurostuff:main Jun 8, 2022
@JulioAPeraza JulioAPeraza deleted the fix-changelog branch June 10, 2022 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improving documentation for the project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants