You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
46 lines
3.1 KiB
46 lines
3.1 KiB
3 weeks ago
|
# Want to contribute?
|
||
|
|
|
||
|
|
If you would like to contribute, here are some notes and guidelines:
|
||
|
|
|
||
|
|
- All new development should be on feature/fix branches, which are then merged to the `master` branch once stable and approved; so the `master` branch is always the most up-to-date, working code
|
||
|
|
- If you are going to submit a pull request, please fork from `master`, and submit your pull request back as a fix/feature branch referencing the GitHub issue number
|
||
|
|
- The code must work with all PHP versions that we support (currently PHP 7.4 to PHP 8.2).
|
||
|
|
- You can call `composer versions` to test version compatibility.
|
||
|
|
- Code style should be maintained.
|
||
|
|
- `composer style` will identify any issues with Coding Style`.
|
||
|
|
- `composer fix` will fix most issues with Coding Style.
|
||
|
|
- All code changes must be validated by `composer check`.
|
||
|
|
- Please include Unit Tests to verify that a bug exists, and that this PR fixes it.
|
||
|
|
- Please include Unit Tests to show that a new Feature works as expected.
|
||
|
|
- Please don't "bundle" several changes into a single PR; submit a PR for each discrete change/fix.
|
||
|
|
- Remember to update documentation if necessary.
|
||
|
|
|
||
|
|
- [Helpful article about forking](https://help.github.com/articles/fork-a-repo/ "Forking a GitHub repository")
|
||
|
|
- [Helpful article about pull requests](https://help.github.com/articles/using-pull-requests/ "Pull Requests")
|
||
|
|
|
||
|
|
## Unit Tests
|
||
|
|
|
||
|
|
When writing Unit Tests, please
|
||
|
|
- Always try to write Unit Tests for both the happy and unhappy paths.
|
||
|
|
- Put all assertions in the Test itself, not in an abstract class that the Test extends (even if this means code duplication between tests).
|
||
|
|
- Include any necessary `setup()` and `tearDown()` in the Test itself.
|
||
|
|
- If you change any global settings (such as system locale, or Compatibility Mode for Excel Function tests), make sure that you reset to the default in the `tearDown()`.
|
||
|
|
- Use the `ExcelError` functions in assertions for Excel Error values in Excel Function implementations.
|
||
|
|
<br />Not only does it reduce the risk of typos; but at some point in the future, ExcelError values will be an object rather than a string, and we won't then need to update all the tests.
|
||
|
|
- Don't over-complicate test code by testing happy and unhappy paths in the same test.
|
||
|
|
|
||
|
|
This makes it easier to see exactly what is being tested when reviewing the PR. I want to be able to see it in the PR, not have to hunt in other unchanged classes to see what the test is doing.
|
||
|
|
|
||
|
|
## How to release
|
||
|
|
|
||
|
|
1. Complete CHANGELOG.md and commit
|
||
|
|
2. Create an annotated tag
|
||
|
|
1. `git tag -a 1.2.3`
|
||
|
|
2. Tag subject must be the version number, eg: `1.2.3`
|
||
|
|
3. Tag body must be a copy-paste of the changelog entries.
|
||
|
|
3. Push the tag with `git push --tags`, GitHub Actions will create a GitHub release automatically, and the release details will automatically be sent to packagist.
|
||
|
|
4. Github seems to remove markdown headings in the Release Notes, so you should edit to restore these.
|
||
|
|
|
||
|
|
> **Note:** Tagged releases are made from the `master` branch. Only in an emergency should a tagged release be made from the `release` branch. (i.e. cherry-picked hot-fixes.)
|
||
|
|
|