Contributions are welcome, and they are greatly appreciated and credit will always be given.
You can contribute in many ways:
Report bugs at https://github.com/riscv/riscof/issues.
The best way to send feedback is to file an issue at https://github.com/riscv/riscof/issues.
If you are proposing a feature:
- Explain in detail how it would work.
- Keep the scope as narrow as possible, to make it easier to implement.
- Remember that this is a volunteer-driven project, and that contributions are welcome :)
The repo adopts a simple git strategy where all contributions to the repo are made to the dev
branch (i.e. all Pull-Requests must use dev
as the target branch). On a monthly cadence (decided
and controlled by the SIG-ARCH-TEST members) the dev
branch will be merged to the main
to by
the official maintainers of the repo. This will create an official release capturing all the
development over the month into a single release.
To implement the above strategy successfully the following needs be followed:
- Developers: All pull-requests from developers must target the
dev
branch and the PR must
contain an entry in the CHANGELOG.md file under [WIP-DEV] section. * Maintainers: When a making a release the maintainers shall assign semantic version number by updating the CHANGELOG and the respective python files before raising a PR from the dev to main.
Ready to contribute? Here's how to set up riscof for local development.
Fork the riscv_ctg repo on GitHub.
Clone your fork locally and checkout the
dev
branch:$ git clone https://github.com/riscv/riscof.git -b dev
Create an issue and WIP merge request that creates a working branch for you:
$ git checkout -b name-of-your-bugfix-or-feature
Now you can make your changes locally.
Commit your changes and push your branch to GitLab:
$ git add . $ git commit -m "Your detailed description of your changes." $ git push origin name-of-your-bugfix-or-feature
6. Submit a pull-request through the GitHub website. Make sure the pull-request is on the dev branch of the origin repo.
7. Do not forget to make an entry in the CHANGELOG.md file under the [WIP-DEV] section highlighting the changes you have done.
Before you submit a merge request, check that it meets these guidelines:
- The merge request should include tests (if any).
- If the merge request adds functionality, the docs should be updated.
- The target branch must always be the dev branch.
When issuing pull requests to the main branch (from dev), a version entry in the CHANGELOG.md is mandatory. The tool adheres to the [Semantic Versioning](https://semver.org/spec/v2.0.0.html) scheme. Following guidelines must be followed while assigning a new version number :
- Patch-updates: all doc updates (like typos, more clarification,etc) and updates to unratified extensions.
- Minor-updates: Fixing bugs in current features, adding new features which do not break current features or working.
- Major-updates: Changes to the framework flow (backward compatible or incompatible).
Note: You can have either a patch or minor or major update. Note: In case of a conflict, the maintainers will decide the final version to be assigned.
To update the version of the python package for deployment you can use bumpversion (installed
using pip install bumpversion
):
$ bumpversion --no-tag --config-file setup.cfg patch # last arg can be: major or minor or patch
If you don't have bumpversion installed you can manually update the version in the following files:
- change the value of variable
current_version
in ./setup.cfg - change the value of variable
__version__
in ./riscof/__init__.py