-
Notifications
You must be signed in to change notification settings - Fork 20
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
Tag creation in kebechet with release option #1058
Tag creation in kebechet with release option #1058
Conversation
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.
can you please address the suggestions and also fix the linting issues
thanks 💯
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.
pre-commit check is failing due to following:
fix end of files.........................................................Failed
- hook id: end-of-file-fixer
- exit code: 1
- files were modified by this hook
Fixing kebechet/managers/version/utils.py
Fixing kebechet/managers/version/version.py
mypy.....................................................................Failed
- hook id: mypy
- exit code: 1
kebechet/managers/version/utils.py:223: error: Item "None" of "Optional[Match[Any]]" has no attribute "group"
Found 1 error in 1 file (checked 37 source files)
black....................................................................Failed
- hook id: black
- files were modified by this hook
reformatted kebechet/managers/version/utils.py
reformatted kebechet/managers/version/version.py
this can be viewed from details button on the failed check
please address the following
Hey @shreekarSS, to comply with formatting standards you can run the follwing commands:
To have this run on your local repo before every commit you can run |
_get_merge_commit_sha() -> to get merge commit sha _get_version() -> Get version from PR branch _is_release_version_pr -> Check if PR is open khebhut bot and is release PR or not Signed-off-by: Shreekar <sshreeka@redhat.com>
5af454e
to
da40513
Compare
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.
one more suggestion
da40513
to
37a3cdf
Compare
Signed-off-by: Shreekar <sshreeka@redhat.com>
37a3cdf
to
a3edba2
Compare
Signed-off-by: Shreekar <sshreeka@redhat.com>
Signed-off-by: Shreekar <sshreeka@redhat.com>
d861dd6
to
65f2936
Compare
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.
/lgtm
thanks 💯
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: harshad16 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Related Issues and Dependencies
Include tag creation in kebechet with release option [https://github.com//issues/1056]
…
This introduces a breaking change
This Pull Request implements
Creation a tag on the merge of the release pull request created by kebechet.