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 PR gating guideline #9611

Merged
merged 1 commit into from
Oct 4, 2022
Merged

Conversation

matejak
Copy link
Member

@matejak matejak commented Oct 3, 2022

Add a section related to gating in order to support the review and merge process.

@matejak matejak added this to the 0.1.65 milestone Oct 3, 2022
@github-actions
Copy link

github-actions bot commented Oct 3, 2022

Start a new ephemeral environment with changes proposed in this pull request:

Fedora Environment
Open in Gitpod

Oracle Linux 8 Environment
Open in Gitpod

@Mab879 Mab879 self-assigned this Oct 3, 2022
@Mab879 Mab879 added the Documentation Update in project documentation. label Oct 3, 2022
Copy link
Member

@Mab879 Mab879 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR. One minor sugguestion. Also please look at the Code Climate changes they seem to make sense to me.

Such failing tests should be addressed in a reviewer's comment to waive them.

Noteworthy additions to these principles:
* CodeClimate: Failures should be taken seriously, especially when they can be fixed easily --- that includes issues related to the complexity of code.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* CodeClimate: Failures should be taken seriously, especially when they can be fixed easily --- that includes issues related to the complexity of code.
* CodeClimate: Failures should be taken seriously, especially when they can be fixed easily — that includes issues related to the complexity of code.

to support the review and merge process.
@codeclimate
Copy link

codeclimate bot commented Oct 4, 2022

Code Climate has analyzed commit b556ddd and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 40.7% (0.0% change).

View more on Code Climate.

@Mab879 Mab879 merged commit a45b876 into ComplianceAsCode:master Oct 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Update in project documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants