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

Improve available CONTRIBUTING information and guidelines #3830

Closed
wants to merge 3 commits into from

Conversation

jrfnl
Copy link
Contributor

@jrfnl jrfnl commented May 19, 2023

Description

CONTRIBUTING: move file to .github directory

... which is nowadays the more common location for this type of file.

Includes updating the link in the README file.

Ref: https://docs.github.com/en/communities/setting-up-your-project-for-healthy-contributions/setting-guidelines-for-repository-contributors

Composer: add scripts for contributors

These scripts should make contributing to PHPCS more straight-forward as they document common CI checks and allow for running these locally.

CONTRIBUTING: rewrite the contributing guide

This is a rewrite of the CONTRIBUTING.md file to document expectations and opportunities for contributors.

It also answers some common questions previously received from contributors.

Suggested changelog entry

N/A

Related issues/external references

N/A

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
    • This change is only breaking for integrators, not for external standards or end-users.
  • Documentation improvement

Copy link
Contributor

@fredden fredden 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 this. The new guide seems clear and has a lot of good advice. 👍

[nit] I noticed two syntax mistakes while reading through.

.github/CONTRIBUTING.md Outdated Show resolved Hide resolved
.github/CONTRIBUTING.md Outdated Show resolved Hide resolved
@jrfnl jrfnl force-pushed the feature/contributing-improvements branch from f34c017 to 1ce2c4e Compare May 19, 2023 12:29
@fredden fredden mentioned this pull request May 26, 2023
12 tasks
.github/CONTRIBUTING.md Outdated Show resolved Hide resolved
@jrfnl jrfnl force-pushed the feature/contributing-improvements branch from 1ce2c4e to 806f69a Compare May 27, 2023 09:56
@jrfnl jrfnl mentioned this pull request Aug 2, 2023
Copy link

@DannyvdSluijs DannyvdSluijs left a comment

Choose a reason for hiding this comment

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

I've been reading through the updated Markdown and it all seems good. Links seem to work, wording is clear and instructive.

@DannyvdSluijs
Copy link

Even though I've approved the PR I now realise this MR will probably break the link to the Contributing.md mentioned in the README.md (see https://github.com/squizlabs/PHP_CodeSniffer#contributing)

... which is nowadays the more common location for this type of file.

Includes updating the link in the README file.

Ref: https://docs.github.com/en/communities/setting-up-your-project-for-healthy-contributions/setting-guidelines-for-repository-contributors
These scripts should make contributing to PHPCS more straight-forward as they document common CI checks and allow for running these locally.
This is a rewrite of the `CONTRIBUTING.md` file to document expectations and opportunities for contributors.

It also answers some common questions previously received from contributors.
@jrfnl jrfnl force-pushed the feature/contributing-improvements branch from 806f69a to 25ab851 Compare September 5, 2023 13:44
@jrfnl
Copy link
Contributor Author

jrfnl commented Sep 5, 2023

Even though I've approved the PR I now realise this MR will probably break the link to the Contributing.md mentioned in the README.md (see https://github.com/squizlabs/PHP_CodeSniffer#contributing)

Good catch! Thanks for that. I've fixed that up in the commit which moves the file now.

@jrfnl
Copy link
Contributor Author

jrfnl commented Oct 28, 2023

Note: depending on which PR gets merged first, this PR will need an update for the changes in #3803.

@jrfnl
Copy link
Contributor Author

jrfnl commented Dec 2, 2023

Closing as replaced by (merged PR) PHPCSStandards/PHP_CodeSniffer#40

@jrfnl jrfnl closed this Dec 2, 2023
@jrfnl jrfnl deleted the feature/contributing-improvements branch December 2, 2023 02:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants