From 9840656c59733047c7ad4d46c766aa3f81f511fb Mon Sep 17 00:00:00 2001 From: Jay Date: Sat, 28 May 2022 17:51:14 +0200 Subject: [PATCH] docs: updated contributing guide to no longer contain the release section as this is not needed --- COLLABORATOR_GUIDE.md | 15 ++++++++------- CONTRIBUTING.md | 9 +-------- 2 files changed, 9 insertions(+), 15 deletions(-) diff --git a/COLLABORATOR_GUIDE.md b/COLLABORATOR_GUIDE.md index 973bc67..80405e9 100644 --- a/COLLABORATOR_GUIDE.md +++ b/COLLABORATOR_GUIDE.md @@ -9,13 +9,14 @@ As a collaborator, you will be involved with axios with some administrative resp 1. __Answer questions.__ It is not expected that you provide answers to questions that aren’t relevant, nor do you need to mentor people on how to use JavaScript, etc. If the question is not directly about the module, please close the issue. If the question stems from poor documentation, please update the docs and consider adding a code example. In any event try to be helpful and remember that there’s no such thing as a stupid question. 1. __Assist with PRs.__ By encouraging contributors to supply a PR for their own issue this is ideally where most of your attention will be focused. Keep a few things in mind as you review PRs. - - When fixing a bug: does the PR adequately solve the problem without introducing any regressions? - - When implementing a feature: does the feature fit within the scope of axios? - - When removing functionality: is it properly deprecated with a warning? - - When introducing functionality: is the API predictable? - - Does the new code work for all supported platforms/browsers? - - Do the tests and linting pass CI? - - Are there tests to validate the changes that have been made? + +- When fixing a bug: does the PR adequately solve the problem without introducing any regressions? +- When implementing a feature: does the feature fit within the scope of axios? +- When removing functionality: is it properly deprecated with a warning? +- When introducing functionality: is the API predictable? +- Does the new code work for all supported platforms/browsers? +- Do the tests and linting pass CI? +- Are there tests to validate the changes that have been made? 1. __Fix bugs and implement features.__ When things need to be fixed or implemented, and a PR can’t wait, you may do things yourself. You should still submit a PR yourself and get it checked off by at least one other contributor. Keep the points from number 4 in consideration as you push your code. diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index ba3c117..51d164b 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -12,7 +12,7 @@ Please follow [conventional commits](https://www.conventionalcommits.org/en/v1.0 ## Testing -Please update the tests to reflect your code changes. Pull requests will not be accepted if they are failing on [Travis CI](https://travis-ci.org/axios/axios). +Please update the tests to reflect your code changes. Pull requests will not be accepted if they are failing on GitHub actions. ## Documentation @@ -28,13 +28,6 @@ Please update the [docs](README.md) accordingly so that there are no discrepanci Please don't include changes to `dist/` in your pull request. This should only be updated when releasing a new version. -## Releasing - -Releasing a new version is mostly automated. For now the [CHANGELOG](https://github.com/axios/axios/blob/master/CHANGELOG.md) requires being updated manually. Once this has been done run the commands below. Versions should follow [semantic versioning](http://semver.org/). - -- `npm version -m "Releasing %s"` -- `npm publish` - ## Running Examples Examples are included in part to allow manual testing.