-
Notifications
You must be signed in to change notification settings - Fork 1
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
upgrade packages and remove AUTHOR_EMAIL and AUTHOR_NAME because it breaks signed commit #17
base: master
Are you sure you want to change the base?
Conversation
@@ -138,16 +136,6 @@ def is_test_env(env: str) -> bool: | |||
else int(os.environ["GITLAB_GRAPHQL_PAGE_LENGTH"]) | |||
) | |||
GITLAB_GROUP_IDENTIFIER = os.getenv("GITLAB_GROUP_IDENTIFIER", "") | |||
GITLAB_AUTHOR_EMAIL = ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'aurais été de l'avis de rajouter une option signed_commit
, qui sera en mutual exclusive avec l'author.
Ca eviterait un changement cassant, mais en temporaire ca me va de simplement degager les 2 options.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree that it’s important to apply conditions in this case, as it remains relevant in other setups as well. I'm working on proposing a change that aligns with this approach.
Co-authored-by: Lowaiz <[email protected]>
No description provided.