Skip to content

Commit

Permalink
Initial version of Best Practices
Browse files Browse the repository at this point in the history
- Includes a simple PR template
  • Loading branch information
jafeltra committed Jun 18, 2024
1 parent dc139e8 commit 1697d50
Show file tree
Hide file tree
Showing 3 changed files with 52 additions and 1 deletion.
5 changes: 5 additions & 0 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
**Description:**

**Testing Instructions:**

**Related Issues:**
46 changes: 46 additions & 0 deletions BEST-PRACTICES.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,46 @@
# Best Practices

We're glad you're considering contributing to SUSHI! Below are a few best practices that we recommend for all contributions.

## Before contributing

Before contributing a feature or a bugfix, we recommend creating a GitHub issue if one does not exist. This allows the community to provide feedback on why an issue may be occurring or provide additional insight into a suggested feature.

If a GitHub issue already exists for what you are planning to contribute, we recommend commenting on the issue to indicate that you are working on an implementation to avoid duplication of work.

## Coding practices

We recommend the following coding practices for high quality contributions:

- Make all changes in a personal [fork](https://help.github.com/articles/fork-a-repo/) of this repository.
- Use descriptive commit messages.
- Prefer self-explanatory code as much as possible, but provide helpful comments for complex expressions and code blocks.
- Add unit tests for any new or changed functionality, and update any existing tests that are impacted by your changes.
- SUSHI uses [Jest](https://jestjs.io/) as a testing framework.
- To run the full test suite, run `npm test`. Ensure all tests are passing.
- Follow the code style and conventions as enforced by the lint configuration and as evidenced by the existing code.
- SUSHI uses [ESLint](https://eslint.org/) for code linting.
- To run the linter on all code, run `npm run lint`. Ensure there are no issues reported.
- Follow the code formatting as enforced by the formatter configuration.
- SUSHI uses [Prettier](https://prettier.io/) for code formating.
- To run the prettier on all code, run `npm run prettier`. Ensure there are no issues reported.
- Ensure any new dependencies do not contain any known security vulnerabilities
- To check for known security vulnerabilities, we recommend using [npm-audit](https://fshschool.org/). Run `npm audit` and ensure there are no new issues on your branch.
- Update documentation to reflect any user-facing changes.
- Documentation updates may include, but are not limited to, the project [README](README.md) and [FSH School](https://fshschool.org/).
- If changes are required to FSH School, following instructions for contributing in the [project repository](https://github.com/FSHSchool/site).

## Making a Pull Request

We recommend the following best practices for creating a high quality pull request:

- Review your own PR before marking it as ready for review by others. Ensure the only code changes included are ones relevant to the feature or bugfix and that they follow the coding practices outlined above.
- Ensure your branch is up to date with master. There are a few ways you can update your branch:
- Use the "Update branch" button available once you make your PR. This is the recommended approach if you are not comfortable with merging or rebasing.
- [Merge](https://git-scm.com/book/en/v2/Git-Branching-Basic-Branching-and-Merging) master into your branch.
- [Rebase](https://git-scm.com/book/en/v2/Git-Branching-Rebasing) your branch on master. We only recommend this approach if you are very comfortable with rebasing.
- Follow the pull request template to create a detailed PR description.
- Include a detailed description of the changes made in the PR.
- Include instructions for how to test the PR. You may want to include a link to sample FSH in FSH Online to demonstrate a bug or attach a sample project that highlights the new or improved behavior.
- [Link the issue](https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue) that the PR addresses.
- Follow up on any discussion on your PR. If changes are requested, make any necessary updates and comment indicating your PR is ready for re-review.
2 changes: 1 addition & 1 deletion CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ This reduces the likelihood of duplicated effort and also provides the maintaine

### Coding Practices

Code that is contributed to this project should be done in a personal fork of this repository and follow the coding practices specified in our Best Practices documentation (coming soon!).
Code that is contributed to this project should be done in a personal fork of this repository and follow the coding practices specified in our Best Practices documentation in [BEST-PRACTICES.md](BEST-PRACTICES.md).

### Before Submitting a Pull Request

Expand Down

0 comments on commit 1697d50

Please sign in to comment.