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

ci: adding workflow for pr conventions #2089

Merged
merged 2 commits into from
Jul 3, 2024
Merged

ci: adding workflow for pr conventions #2089

merged 2 commits into from
Jul 3, 2024

Conversation

hamza-m-masood
Copy link
Contributor

Which problem does the PR fix?

related: #1581

What's in this PR?

Enforcing PR conventions

Checklist

Please make sure to follow our Contributing Guide.

Before opening the PR:

  • In the repo's root dir, run make go.update-golden-only.
  • There is no other open pull request for the same update/change.
  • Tests for charts are added (if needed).
  • In-repo documentation are updated (if needed).

After opening the PR:

  • Did you sign our CLA (Contributor License Agreement)? It will show once you open the PR.
  • Did all checks/tests pass in the PR?

@hamza-m-masood hamza-m-masood changed the title adding workflow for pr conventions ci: adding workflow for pr conventions Jun 27, 2024
@hamza-m-masood hamza-m-masood marked this pull request as ready for review June 27, 2024 21:55
@hamza-m-masood hamza-m-masood self-assigned this Jun 27, 2024
@hamza-m-masood hamza-m-masood added this to the 8.6 Release Cycle milestone Jun 27, 2024
@hamza-m-masood hamza-m-masood linked an issue Jun 27, 2024 that may be closed by this pull request
@hamza-m-masood hamza-m-masood enabled auto-merge (squash) June 27, 2024 21:56
auto-merge was automatically disabled July 2, 2024 16:30

Pull request was closed

@hamza-m-masood hamza-m-masood reopened this Jul 2, 2024
@hamza-m-masood
Copy link
Contributor Author

Interestingly when I only ahve one commit I get this error:

Error: Pull request has only one commit and it's not semantic; this may lead to a non-semantic commit in the base branch (see https://github.com/community/community/discussions/16271). Amend the commit message to match the pull request title, or add another commit.

@@ -0,0 +1,34 @@
name: Pull Request Conventions
Copy link
Member

Choose a reason for hiding this comment

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

Following the same convention let's call it Repo - Pull Request Conventions

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

Copy link
Member

@aabouzaid aabouzaid left a comment

Choose a reason for hiding this comment

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

LGTM, thank you 👍

@hamza-m-masood hamza-m-masood merged commit 7972755 into main Jul 3, 2024
3 checks passed
@hamza-m-masood hamza-m-masood deleted the pr-conventions branch July 3, 2024 16:26
aabouzaid pushed a commit that referenced this pull request Jul 13, 2024
* ci: adding workflow for pr conventions
aabouzaid pushed a commit that referenced this pull request Jul 13, 2024
* ci: adding workflow for pr conventions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[TASK] Enforce conventional commits
2 participants