mirror of
https://github.com/JamesIves/github-pages-deploy-action.git
synced 2023-12-15 20:03:39 +08:00
Update CONTRIBUTING.md
This commit is contained in:
parent
cd57aa99e2
commit
a3009d5743
@ -1,18 +1,19 @@
|
|||||||
# Contributing
|
# Contributing ✏️
|
||||||
|
|
||||||
When contributing to this repository, please first discuss the change you wish to make via issue,
|
When contributing to this repository, please first discuss the change you wish to make via issue,
|
||||||
email, or any other method with the owners of this repository before making a change.
|
[email, or any other method with the owners of this repository](https://jamesiv.es) before making a change.
|
||||||
|
|
||||||
## Before Making a Pull Request
|
## Before Making a Pull Request 🎒
|
||||||
|
|
||||||
1. Ensure that you've tested your feature/change yourself. As the primary focus of this project is deployment, providing a link to a deployed repository using your branch is preferred. You can reference the forked action using your GitHub username, for example `yourname/github-pages-deplpy-action@master`.
|
1. Ensure that you've tested your feature/change yourself. As the primary focus of this project is deployment, providing a link to a deployed repository using your branch is preferred. You can reference the forked action using your GitHub username, for example `yourname/github-pages-deplpy-action@master`.
|
||||||
2. Make sure you update the README if you've made a change that requires documentation.
|
2. Ensure your change passes all of the integration tests.
|
||||||
3. When making a pull request, highlight any areas that may cause a breaking change so the maintainer can update the version number accordingly on the GitHub marketplace.
|
3. Make sure you update the README if you've made a change that requires documentation.
|
||||||
4. Make sure you've formatted and linted your code. You can do this by running `yarn format` and `yarn lint`.
|
4. When making a pull request, highlight any areas that may cause a breaking change so the maintainer can update the version number accordingly on the GitHub marketplace and package registries.
|
||||||
5. Fix or add any tests where applicable. You can run `yarn test` to run the suite. As this action is small in scope it's important that a high level of test coverage is maintained. All tests are written using Jest.
|
5. Make sure you've formatted and linted your code. You can do this by running `yarn format` and `yarn lint`.
|
||||||
6. Ensure all typing is accurate and the action compiles correctly by running `yarn build`.
|
6. Fix or add any tests where applicable. You can run `yarn test` to run the suite. As this action is small in scope it's important that a high level of test coverage is maintained. All tests are written using [Jest](https://jestjs.io/).
|
||||||
|
7. As this package is written in [TypeScript](https://www.typescriptlang.org/) please ensure all typing is accurate and the action compiles correctly by running `yarn build`.
|
||||||
|
|
||||||
# Deploying
|
## Deploying 🚚
|
||||||
|
|
||||||
In order to deploy and test your own fork of this action, you must commit the `node_modules` dependencies. Be sure to run `nvm use` before installing any dependencies. You can learn more about nvm [here](https://github.com/nvm-sh/nvm/blob/master/README.md).
|
In order to deploy and test your own fork of this action, you must commit the `node_modules` dependencies. Be sure to run `nvm use` before installing any dependencies. You can learn more about nvm [here](https://github.com/nvm-sh/nvm/blob/master/README.md).
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user