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.

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

Add auto-author and auto-label to github workflow #65

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Add auto-author and auto-label to github workflow #65

wants to merge 4 commits into from

Conversation

RayStick
Copy link
Member

@RayStick RayStick commented Apr 24, 2024 •
edited

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

@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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@smoia smoia Awaiting requested review from smoia

Assignees

@RayStick RayStick

Labels
Internal Changes affect the internal API. It doesn't increase the version, but produces a changelog
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Implement auto-author-assign and auto-label workflows
2 participants
@RayStick @maestroque