Skip to content

Commit

Permalink
Fix wording and document references
Browse files Browse the repository at this point in the history
Ref. #660
  • Loading branch information
Alexander Senier committed Jul 13, 2021
1 parent 9c55030 commit 7d41dbc
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -296,7 +296,7 @@ $ make install_gnatstudio

Contributions and feedback to RecordFlux are very welcome. To discuss a bug or an enhancement, [open a ticket on GitHub](https://github.com/Componolit/RecordFlux/issues/new/choose) and select the appropriate issue template. Please give sufficient information about your issue, the software version you are using and your environment such that the developers can understand and (if necessary) reproduce the problem. If none of the provided issue templates fit your needs, feel free to open [a blank issue](https://github.com/Componolit/RecordFlux/issues/new).

See [`doc/Development.md`](/doc/Development.md) on how to contribute to RecordFlux.
See [development documentation](/doc/Development.md) on how to contribute to RecordFlux.

## Limitations

Expand Down
4 changes: 2 additions & 2 deletions doc/Development.md
Original file line number Diff line number Diff line change
Expand Up @@ -24,10 +24,10 @@ Additional tools can be found in `tools/`.

## Pull requests

We accept pull requests [via GitHub](https://github.com/Componolit/RecordFlux/compare). To contribute to the project, fork it under your own GitHub user and perform your changes on a topic branch. Ideally, create an issue in the upstream repository describing the problem you would like to solve and your intention to work on it. This will help us to point you to potential prior work and discuss your idea. Your branch should be named `issue_<ISSUE_NUMBER>`, e.g. `issue_694` where #694 is the ticket you created. For small (!) changes descriptive branch names without a ticket are acceptable. We usually prefix those with our GitHub username (e.g. `@senier_some_small_fix`) to be able to easily identify the originator of a branch.
We accept pull requests [via GitHub](https://github.com/Componolit/RecordFlux/compare). To contribute to the project, fork it under your own GitHub user and perform your changes on a topic branch. Ideally, create an issue in the upstream repository describing the problem you would like to solve and your intention to work on it. This will help us to point you to potential prior work and discuss your idea. Your branch should be named `issue_<ISSUE_NUMBER>`, e.g. `issue_694` where #694 is the ticket you created. For small (!) changes descriptive branch names without a ticket are acceptable.

When submitting a pull request, your topic branch should be rebased to the current upstream `main` branch. Verify that all automatic checks performed by `make check`, `make test` and `make prove` succeed before submitting the PR. For Python code we follow and automatically enforce the coding style of [Black](https://pypi.org/project/black/). You can format your code automatically using the `make format` target on the command line. For Ada code (examples as well as generated code) please follow [our Ada style guide](https://github.com/Componolit/ada-style).

We enforce 100% statement coverage for Python code using [pytest](https://pytest.org). Make sure to add relevant test cases to achieve that for your code. See `tests/README.md` and have a look at the existing test cases in the `tests` directory to get an idea of the structure of our test suite. Our Python code is also statically type-checked using [mypy](http://mypy-lang.org/). Make sure to include the required type annotations with your code.
We enforce 100% branch coverage for Python code using [pytest](https://pytest.org). Make sure to add relevant test cases to achieve that for your code. See [the test documentation](/tests/README.md) and have a look at the existing test cases in the `tests` directory to get an idea of the structure of our test suite. Our Python code is also statically type-checked using [mypy](http://mypy-lang.org/). Make sure to include the required type annotations with your code.

Your code will be reviewed by at least one core developer before inclusion into the project. Don't be discouraged should we have many comments and ask you for a lot of changes to your pull request. This even happens to the most experienced developers in our project and we consider these discussions an essential part of the development process and a necessity to maintain high quality. Don't hesitate to open an issue if you have any question or submit the pull request in draft mode first.

0 comments on commit 7d41dbc

Please sign in to comment.