From 7f192e73d38d6e80023bcda73f6fc4c503a5b1ef Mon Sep 17 00:00:00 2001 From: Anush Kumar Date: Thu, 21 Aug 2025 16:05:01 -0700 Subject: [PATCH 1/4] docs(contributing): added PR title format --- docs/CONTRIBUTING.md | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/docs/CONTRIBUTING.md b/docs/CONTRIBUTING.md index a573713a4e6d88..dc03192ef6a38d 100644 --- a/docs/CONTRIBUTING.md +++ b/docs/CONTRIBUTING.md @@ -29,7 +29,8 @@ If you have a substantial feature or a design discussion that you'd like to have Before you submit your Pull Request (PR), consider the following guidelines: - Search GitHub for an open or closed PR that relates to your submission. You don't want to duplicate effort. -- Follow the [standard GitHub approach](https://help.github.com/en/github/collaborating-with-issues-and-pull-requests/creating-a-pull-request-from-a-fork) to create the PR. Please also follow our [commit message format](#commit-message-format). +- Open a pull request (PR) following [GitHub’s standard workflow](https://help.github.com/en/github/collaborating-with-issues-and-pull-requests/creating-a-pull-request-from-a-fork). +- Make sure your PR title and commit messages follow our [commit message format](#commit-message-format) for clarity and consistency. - If there are any breaking changes, potential downtime, deprecations, or big feature please add an update in [Updating DataHub under Next](how/updating-datahub.md). - That's it! Thank you for your contribution! From 9d7a184acf1f63ea7890c7beea600c258f3feb18 Mon Sep 17 00:00:00 2001 From: Anush Kumar Date: Thu, 21 Aug 2025 16:34:55 -0700 Subject: [PATCH 2/4] Update CONTRIBUTING.md --- docs/CONTRIBUTING.md | 65 +++++++++----------------------------------- 1 file changed, 13 insertions(+), 52 deletions(-) diff --git a/docs/CONTRIBUTING.md b/docs/CONTRIBUTING.md index dc03192ef6a38d..471f5fc6667f7c 100644 --- a/docs/CONTRIBUTING.md +++ b/docs/CONTRIBUTING.md @@ -6,11 +6,11 @@ We always welcome contributions to help make DataHub better. Take a moment to re Have ideas about how to make DataHub better? Head over to [DataHub Feature Requests](https://feature-requests.datahubproject.io/) and tell us all about it! -Show your support for other requests by upvoting; stay up to date on progess by subscribing for updates via email. +Show your support for other requests by upvoting; stay up to date on progress by subscribing for updates via email. ## Reporting Issues -We use GitHub issues to track bug reports and submitting pull requests. +We use GitHub issues to track bug reports and submit pull requests. If you find a bug: @@ -22,7 +22,7 @@ If you find a bug: ## Submitting a Request For Comment (RFC) -If you have a substantial feature or a design discussion that you'd like to have with the community follow the RFC process outlined [here](./rfc.md) +If you have a substantial feature or a design discussion that you'd like to have with the community, follow the RFC process outlined [here](./rfc.md). ## Submitting a Pull Request (PR) @@ -30,27 +30,22 @@ Before you submit your Pull Request (PR), consider the following guidelines: - Search GitHub for an open or closed PR that relates to your submission. You don't want to duplicate effort. - Open a pull request (PR) following [GitHub’s standard workflow](https://help.github.com/en/github/collaborating-with-issues-and-pull-requests/creating-a-pull-request-from-a-fork). -- Make sure your PR title and commit messages follow our [commit message format](#commit-message-format) for clarity and consistency. -- If there are any breaking changes, potential downtime, deprecations, or big feature please add an update in [Updating DataHub under Next](how/updating-datahub.md). +- Please make sure to follow our [PR Title Format](#pr-title-format) for clarity and consistency. +- PRs are squashed and merged, resulting in a single commit with the PR title as the commit message. +- If there are any breaking changes, potential downtime, deprecations, or big features, please add an update in [Updating DataHub under Next](how/updating-datahub.md). - That's it! Thank you for your contribution! -## Commit Message Format - -Please follow the [Conventional Commits](https://www.conventionalcommits.org/) specification for the commit message format. In summary, each commit message consists of a _header_, a _body_ and a _footer_, separated by a single blank line. - +### PR Title Format ``` [optional scope]: - -[optional body] - -[optional footer(s)] ``` +Example: +``` +feat(parser): add ability to parse arrays +``` +#### Type -Any line of the commit message cannot be longer than 88 characters! This allows the message to be easier to read on GitHub as well as in various Git tools. - -### Type - -Must be one of the following (based on the [Angular convention](https://github.com/angular/angular/blob/22b96b9/CONTRIBUTING.md#-commit-message-guidelines)): +Must be one of the following: - _feat_: A new feature - _fix_: A bug fix @@ -61,37 +56,3 @@ Must be one of the following (based on the [Angular convention](https://github.c - _style_: Changes that do not affect the meaning of the code (whitespace, formatting, missing semicolons, etc.) - _build_: Changes that affect the build system or external dependencies - _ci_: Changes to our CI configuration files and scripts - -A scope may be provided to a commit’s type, to provide additional contextual information and is contained within parenthesis, e.g., - -``` -feat(parser): add ability to parse arrays -``` - -### Description - -Each commit must contain a succinct description of the change: - -- use the imperative, present tense: "change" not "changed" nor "changes" -- don't capitalize the first letter -- no dot(.) at the end - -### Body - -Just as in the description, use the imperative, present tense: "change" not "changed" nor "changes". The body should include the motivation for the change and contrast this with previous behavior. - -### Footer - -The footer should contain any information about _Breaking Changes_, and is also the place to reference GitHub issues that this commit _Closes_. - -_Breaking Changes_ should start with the words `BREAKING CHANGE:` with a space or two new lines. The rest of the commit message is then used for this. - -### Revert - -If the commit reverts a previous commit, it should begin with `revert:`, followed by the description. In the body it should say: `Refs: ...`, where the hashs are the SHA of the commits being reverted, e.g. - -``` -revert: let us never again speak of the noodle incident - -Refs: 676104e, a215868 -``` From 0e99a4d304fb8b72fe0915ce58695cebf5a89551 Mon Sep 17 00:00:00 2001 From: Anush Kumar Date: Thu, 21 Aug 2025 16:36:42 -0700 Subject: [PATCH 3/4] Update pull_request_template.md --- .github/pull_request_template.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index 699991a1eea120..89366cc1d95539 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -4,7 +4,7 @@ Thank you for contributing to DataHub! Before you submit your PR, please go through the checklist below: -- [ ] The PR conforms to DataHub's [Contributing Guideline](https://github.com/datahub-project/datahub/blob/master/docs/CONTRIBUTING.md) (particularly [Commit Message Format](https://github.com/datahub-project/datahub/blob/master/docs/CONTRIBUTING.md#commit-message-format)) +- [ ] The PR conforms to DataHub's [Contributing Guideline](https://github.com/datahub-project/datahub/blob/master/docs/CONTRIBUTING.md) (particularly [PR Title Format](https://github.com/datahub-project/datahub/blob/master/docs/CONTRIBUTING.md#pr-title-format)) - [ ] Links to related issues (if applicable) - [ ] Tests for the changes have been added/updated (if applicable) - [ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same. From 6f6098b8914d7981e70ec13a00035ce1b4523422 Mon Sep 17 00:00:00 2001 From: Anush Kumar Date: Thu, 21 Aug 2025 17:05:07 -0700 Subject: [PATCH 4/4] style: file lint fix --- docs/CONTRIBUTING.md | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/docs/CONTRIBUTING.md b/docs/CONTRIBUTING.md index 471f5fc6667f7c..43cc8721602a4f 100644 --- a/docs/CONTRIBUTING.md +++ b/docs/CONTRIBUTING.md @@ -36,13 +36,17 @@ Before you submit your Pull Request (PR), consider the following guidelines: - That's it! Thank you for your contribution! ### PR Title Format + ``` [optional scope]: ``` + Example: + ``` feat(parser): add ability to parse arrays ``` + #### Type Must be one of the following: