Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Updating contribution #363

Merged
merged 7 commits into from
Sep 6, 2023
Merged

Updating contribution #363

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
b7e19fe
Updating contribution with jenkins changes
Nazim-crim Aug 9, 2023
e079520
Updating PR template
Nazim-crim Aug 11, 2023
ab0662f
Updating contribution with new variable name and PR template with add…
Nazim-crim Aug 17, 2023
17a02bb
Merge branch 'master' of github.com:bird-house/birdhouse-deploy into …
Nazim-crim Aug 25, 2023
e856903
Updating contribution
Nazim-crim Aug 31, 2023
619cdc6
Merge branch 'master' of github.com:bird-house/birdhouse-deploy into …
Nazim-crim Aug 31, 2023
3ebd044
Merge branch 'master' into jenkins-contribution
Nazim-crim Sep 6, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,3 +25,11 @@ List any dependencies that are required for this change.
Links to other issues or sources.

- [ ] Things to do...

<!--
The test suite can be run using a different DACCS config with ``birdhouse_daccs_configs_branch: branch_name`` in the PR description.
To globally skip the test suite regardless of the commit message use ``birdhouse_skip_ci: true`` in the PR description.
-->

birdhouse_daccs_configs_branch: master
birdhouse_skip_ci: false
9 changes: 7 additions & 2 deletions CONTRIBUTING.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,12 +37,17 @@ Policy rules

3. Contributions will trigger a test suite that must successfully pass before being merged (or integrated).

* The test suite can be run using a different DACCS config with ``birdhouse_daccs_configs_branch: branch_name`` in the PR description.
* It is possible to skip the test suite if the latest commit contains either ``[skip ci]``, ``[ci skip]`` or ``[no ci]``.
To globally skip the test suite regardless of the commit message use ``birdhouse_skip_ci: true`` in the PR description.

4. Contributions must be reviewed by every willing organizations
(Default reviewers are `@tlvu`_ for `Ouranos`_ and `@MatProv`_ for `CRIM`_).
(Default reviewers are `@mishaschwartz`_ for `UofT` , `@tlvu`_ for `Ouranos`_ and `@fmigneault`_ for `CRIM`_).

.. _@mishaschwartz: https://github.com/mishaschwartz
.. _@tlvu: https://github.com/tlvu
.. _Ouranos: https://github.com/Ouranosinc
.. _@MatProv: https://github.com/MatProv
.. _@fmigneault: https://github.com/fmigneault
.. _CRIM: https://github.com/crim-ca

5. The reviews must be rigorous while respecting the initial scope.
Expand Down
Loading