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 auto-author and auto-label to github workflow #65

Merged
merged 5 commits into from
Jun 6, 2024
Merged

Conversation

RayStick
Copy link
Member

@RayStick RayStick commented Apr 24, 2024

Closes #64

Proposed Changes

  • The same as this PR for phys2bids but implemented for this repo
  • All I did was change the directory locations in the .github/labeler.yml file, otherwise the files are the same as the phys2bids files

Change Type

  • bugfix (+0.0.1)
  • minor (+0.1.0)
  • major (+1.0.0)
  • refactoring (no version update)
  • test (no version update)
  • infrastructure (no version update)
  • documentation (no version update)
  • other

Checklist before review

  • I added everything I wanted to add to this PR.
  • [Code or tests only] I wrote/updated the necessary docstrings.
  • [Code or tests only] I ran and passed tests locally.
  • [Documentation only] I built the docs locally.
  • My contribution is harmonious with the rest of the code: I'm not introducing repetitions.
  • My code respects the adopted style, especially linting conventions.
  • The title of this PR is explanatory on its own, enough to be understood as part of a changelog.
  • I added or indicated the right labels.
  • I added information regarding the timeline of completion for this PR.
  • Please, comment on my PR while it's a draft and give me feedback on the development!

@RayStick RayStick self-assigned this Apr 24, 2024
@RayStick RayStick requested a review from smoia April 24, 2024 15:23
@RayStick RayStick added the Internal Changes affect the internal API. It doesn't increase the version, but produces a changelog label Apr 24, 2024
@RayStick
Copy link
Member Author

@smoia ready to review but pre-commit is not happy

@RayStick RayStick marked this pull request as ready for review April 24, 2024 15:56
@maestroque
Copy link
Contributor

@RayStick pre-commit is failing because it is not yet configured for this repository.
After #67 is merged then this could be merged as well, since it's already integrated to phys2bids (although order shouldn't matter).

@RayStick
Copy link
Member Author

RayStick commented Jun 3, 2024

@maestroque ah great, thanks for the context! Will wait for #67 then

@maestroque
Copy link
Contributor

@RayStick #67 is now merged! You could now pull from the master branch, and this should be ready to go

bring branch up to date  with master
@maestroque maestroque self-requested a review June 6, 2024 11:47
@RayStick
Copy link
Member Author

RayStick commented Jun 6, 2024

Thank you @maestroque!

@RayStick RayStick merged commit f103d88 into master Jun 6, 2024
1 check passed
@RayStick RayStick deleted the add-workflows branch June 6, 2024 11:53
@smoia
Copy link
Member

smoia commented Jul 17, 2024

🚀 PR was released in 0.4.0 🚀

@smoia smoia added the released This issue/pull request has been released label Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Internal Changes affect the internal API. It doesn't increase the version, but produces a changelog released This issue/pull request has been released
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Implement auto-author-assign and auto-label workflows
3 participants