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

Change contributing workflow #170

Merged
merged 10 commits into from
Apr 29, 2024

Conversation

sappenin
Copy link
Collaborator

@sappenin sappenin commented Jan 23, 2024

This PR re-organizes the Contributing guidelines to reflect the process agreed upon by @Silkjaer, @intelliot , @sappenin, @amarantha-k and a few others in April of 2023 (but which was never update in this repo).

Given that it has unfortunately been some time since this discussion took place, if you would like to propose any changes to this PR, please feel free.

…XRPL-Standards into df/update-process

# Conflicts:
#	CONTRIBUTING.md
…XRPL-Standards into df/update-process

# Conflicts:
#	.gitignore
…ange-contributing-workflow

# Conflicts:
#	.gitignore
#	README.md
@sappenin sappenin self-assigned this Jan 23, 2024
@sappenin sappenin added the enhancement New feature or request label Jan 23, 2024
@sappenin
Copy link
Collaborator Author

Bumping this once more - I realize these changes are quite old, so am open to feedback, suggestions, debate, etc.

  1. While there are a lot of changes in this PR, most of these changes are simply moving text around for clarity and improved organization.
  2. The existing README.md and CONTRIBUTING.md files in master have similar -- though not always the same -- content, which may cause confusion or ambiguity. This PR prefers to move any content related to contributions into CONTRIBUTING.md, for clarity.
  3. In the intervening months since the changes in this PR were first proposed, some different changes were merged into the README. E.g., see here, here, and here. I have incorporated these into CONTRIBUTING.md in this PR. If I missed anything, please suggest a change. @WietseWind @RichardAH
  4. I have only medium conviction around these changes, so while I think they improve the overall process, I won't be offended if anyone wants to suggest changes to what is proposed here.

@Silkjaer @intelliot

@sappenin sappenin requested review from mvadari and mDuo13 April 10, 2024 21:57
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
Remove period from anchor link

Co-authored-by: Ed Hennis <ed@ripple.com>
@sappenin sappenin requested review from ximinez and mvadari April 16, 2024 23:04
Copy link
Collaborator

@amarantha-k amarantha-k 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 updates, @sappenin! Other than a minor suggestion, looks good.

CONTRIBUTING.md Outdated Show resolved Hide resolved
Co-authored-by: Amarantha Kulkarni <amarantha-k@users.noreply.github.com>
@sappenin sappenin merged commit e6ecd5c into XRPLF:master Apr 29, 2024
@sappenin sappenin deleted the change-contributing-workflow branch April 29, 2024 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants