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

CASSGO-10 Update CONTRIBUTING.md #1835

Merged
merged 1 commit into from
Nov 7, 2024

Conversation

joao-r-reis
Copy link
Contributor

@joao-r-reis joao-r-reis commented Oct 24, 2024

The section for the commit message format is too generic, so this commit adds more details and a clear example of how a commit message should be structured.

This commit also makes it clear in CONTRIBUTING.md that JIRA should be used instead of GH issues and encourages contributors to seek feedback and discussion on the proposed changes early.

@joao-r-reis joao-r-reis changed the title Update CONTRIBUTING.md CASSGO-10 Update CONTRIBUTING.md Oct 24, 2024
CONTRIBUTING.md Outdated

Common ways to engage with the GoCQL community are:

- CASSGO project on ASF JIRA

This comment was marked as resolved.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hmm I'm not sure what hyperlink I should use for the ML, I think I'll use the archives link (https://lists.apache.org/[email protected])

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

CONTRIBUTING.md Outdated
@@ -23,21 +37,72 @@ The following is a check list of requirements that need to be satisfied in order
* Notable changes (i.e. new features or changed behavior, bugfixes) are appropriately documented in CHANGELOG.md, functional changes also in godoc

This comment was marked as resolved.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hmm they are... @michaelsembwever do you have any suggestions regarding this? Is it ok if we just keep using CHANGELOG.md but new entries use JIRA issue IDs instead of GH pull request numbers? Otherwise we'd have to keep CHANGELOG.md around if anyone wants to look at the changelog for older releases and use CHANGES.txt for new releases.

Copy link
Member

Choose a reason for hiding this comment

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

This is up to you to figure out :)

The server provides CHANGES.txt and NEWS.txt. The former is meant to provide outside-in PoV short description to each change (which jira ticket titles and description often fail at), and the latter important must-read info. Drivers don't need to follow this precedent.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh in that case yeah we should probably keep using CHANGELOG.md but I'll see if I can get other opinions on this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated CONTRIBUTING.md so references to CHANGES.txt are replaced with CHANGELOG.md

CONTRIBUTING.md Outdated Show resolved Hide resolved
@joao-r-reis joao-r-reis force-pushed the update-contributing branch 3 times, most recently from 157c6b5 to af779fb Compare October 29, 2024 11:22
CONTRIBUTING.md Outdated
Common ways to engage with the GoCQL community are:

- [CASSGO project on ASF JIRA](https://issues.apache.org/jira/projects/CASSGO/issues/)
- Apache Cassandra dev mailing list - [[email protected]](mailto:[email protected])
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it intended that there is dev-subscribe in the mailto: URL?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nope, good catch. Fixed

The section for the commit message format is too generic,
so this commit adds more details and a clear example of
how a commit message should be structured.

This commit also makes it clear in CONTRIBUTING.md that
JIRA should be used instead of GH issues and encourages
contributors to seek feedback and discussion on the proposed
changes early.

patch by João Reis; reviewed by Mick Semb Wever and Martin Sucha for CASSGO-10
@joao-r-reis joao-r-reis merged commit 47886e5 into apache:trunk Nov 7, 2024
2 checks passed
@joao-r-reis joao-r-reis deleted the update-contributing branch November 7, 2024 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants