@@ -5,6 +5,10 @@ welcome: bug reports, bug fixes, documentation improvements, translations, and n
55
66Please take a moment to read this guide before opening an issue or pull request.
77
8+ > ** Pull requests are restricted to project collaborators.** If you are not a collaborator, please
9+ > [ open an issue] ( https://github.com/tecnickcom/tcexam/issues ) instead of a pull request,
10+ > describing the bug or feature in detail. A maintainer will review it and take it from there.
11+
812## Code of Conduct
913
1014This project follows the [ Contributor Covenant Code of Conduct] ( CODE_OF_CONDUCT.md ) . By
@@ -31,8 +35,13 @@ using the bug-report template and include a **clear title and description**, the
3135
3236## Submitting a bug fix
3337
34- 1 . Fork the repository and create a branch from ` main `
35- (e.g. ` git checkout -b fix/short-description ` ).
38+ > Only project collaborators can open pull requests. If you are not a collaborator, please
39+ > [ open an issue] ( https://github.com/tecnickcom/tcexam/issues/new ) describing the bug in detail
40+ > (see [ Reporting a bug] ( #reporting-a-bug ) ). A maintainer will take it from there.
41+
42+ Collaborators preparing a fix:
43+
44+ 1 . Create a branch from ` main ` (e.g. ` git checkout -b fix/short-description ` ).
36452 . Make your changes, following the existing conventions in the surrounding code.
37463 . Add or update tests to cover the change.
38474 . Run the quality-assurance suite locally and ensure it passes (see below).
@@ -44,8 +53,8 @@ using the bug-report template and include a **clear title and description**, the
4453Before writing any code, ** open a Feature Request** on
4554[ GitHub Issues] ( https://github.com/tecnickcom/tcexam/issues/new ) describing the use case and
4655proposed behaviour, and wait for feedback. This avoids investing time in a direction that may
47- not be accepted. Once agreed, follow the same branch → code → test → PR workflow as for bug fixes
48- (use a branch named ` feature/short-description ` ).
56+ not be accepted. Once agreed, a collaborator will implement it following the same
57+ branch → code → test → PR workflow as for bug fixes (using a branch named ` feature/short-description ` ).
4958
5059## Development workflow
5160
@@ -70,6 +79,12 @@ Database Abstraction Layer or controller logic, also run `make dockertest` (MySQ
7079
7180## Pull request guidelines
7281
82+ > Opening pull requests is restricted to project collaborators. If you are an external
83+ > contributor, please [ open an issue] ( https://github.com/tecnickcom/tcexam/issues/new ) describing
84+ > the problem or feature in detail instead.
85+
86+ - ** Sign the Contributor License Agreement (CLA).** On your first pull request the CLA Assistant
87+ bot will comment with a link to sign; the PR cannot be merged until the CLA is signed.
7388- Target the ` main ` branch and keep PRs focused — one fix or feature per PR.
7489- Ensure ` make qa ` passes locally before opening the PR.
7590- Cover new code with tests and do not let coverage regress.
0 commit comments