Skip to content

Commit 3945baf

Browse files
Closes #2979: update contributions guidelines (#2987)
* #2979 update contributions guidelines * Update .github/CONTRIBUTING.md Co-authored-by: Ben Straub <ben.x.straub@gsk.com> * #2979 update lycheeignore with iso link * #2979 fix casing * #2970 fix title sizes --------- Co-authored-by: Ben Straub <ben.x.straub@gsk.com>
1 parent 1e6d6dd commit 3945baf

File tree

5 files changed

+36
-8
lines changed

5 files changed

+36
-8
lines changed

.github/CONTRIBUTING.md

Lines changed: 31 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,10 @@
11
# Contribution to {admiral}
22

3-
This outlines how to propose a change to the admiral package. For more detailed info about contributing to {admiral}, and other [pharmaverse packages](https://pharmaverse.org/), please see the [Contribution Guide](https://pharmaverse.github.io/admiral/cran-release/CONTRIBUTING.html) as well as other Developer Guides in the Articles section of the [{admiraldev} website](https://pharmaverse.github.io/admiraldev/).
3+
This outlines how to propose a change to the admiral package. For more detailed info about contributing to `{admiral}`, and other [pharmaverse packages](https://pharmaverse.org/), please see the [Contribution Guide](https://pharmaverse.github.io/admiral/cran-release/CONTRIBUTING.html) as well as other Developer Guides in the Articles section of the [{admiraldev} website](https://pharmaverse.github.io/admiraldev/).
44

55
Please note that we try to align to best practices used in other R packages' development processes - so veteran developers should be familiar with our processes. However, we do deviate slightly from some best practices and we advise all new contributors to review our package documentation accordingly.
66

7-
8-
# Basics of Contribution
7+
## Basics of Contribution
98
🦋 For each new contribution, the user creates an issue on the issue tab on [GitHub](https://github.com/pharmaverse/admiral/issues) to put it in our backlog. The issues can range from bug identification and/or fixes, enhancements to functions, documentation, tests or new features.
109

1110
🦋 We advise you to contact us when an [issue](https://github.com/pharmaverse/admiral/issues) is created via [Slack](https://app.slack.com/client/T028PB489D3/C02M8KN8269) (If you don't have access, use this [link](https://join.slack.com/t/pharmaverse/shared_invite/zt-yv5atkr4-Np2ytJ6W_QKz_4Olo7Jo9A) to join). We can discuss details or align expectations if you are not familiar with the `{admiral}` philosophy and programming strategy. The team will try to review the issues within the next backlog meeting and give some initial feedback. Since we are not a 100% fully resourced software development team it might be that some issues will take longer to respond to depending on the amount of overall issues.
@@ -16,24 +15,24 @@ Please note that we try to align to best practices used in other R packages' dev
1615

1716
🦋 Issues are meant to be taken on by users from the Pharma programming community and not exclusively by the `{admiral}` team from GSK and Roche.
1817

19-
# Contribution Model
18+
## Contribution Model
2019

21-
## Type 1: Contribution without Code
20+
### Type 1: Contribution without Code
2221

2322
🦋 User creates an issue and ideally contacts an `{admiral}` team member via [Slack](https://app.slack.com/client/T028PB489D3/C02M8KN8269).
2423

2524
🦋 The `{admiral}` core development team will contact the issue creator as soon as possible to discuss further details.
2625

2726

28-
## Type 2: Contribution with Code
27+
### Type 2: Contribution with Code
2928

3029
🦋 First, the user creates an issue or comments on an existing issue to notify that they’d like to contribute code.
3130

3231
🦋 Follow our development process step-by-step guide.
3332

3433
🦋 We advise to contact an `{admiral}` core development team directly via [Slack](https://app.slack.com/client/T028PB489D3/C02M8KN8269) before submitting code for complex functionality.
3534

36-
## Detailed Development Process
35+
### Detailed Development Process
3736

3837
If you decide to contribute with code and you're ready to make your first code contribution, this detailed development process step-by-step guide will help tie all the other detailed vignettes together to give you the simplest experience of helping to grow and enhance our codebase.
3938

@@ -96,3 +95,28 @@ reviews are for. Also if ever you get stuck don't hesitate to reach out
9695
for support via the [Slack
9796
channel](https://pharmaverse.slack.com/).
9897
***Welcome to our `{admiral}` community!***
98+
99+
## Acknowledgements for Contributions
100+
101+
The `{admiral}` team has several ways to acknowledge individuals who have contributed to `{admiral}` or related packages:
102+
103+
🦋 **Author (i.e. individual listed on [DESCRIPTION](https://github.com/pharmaverse/admiral/blob/main/DESCRIPTION) file as author)**: This is for
104+
individuals who are currently part of the development team for the package and are making sustained contributions over time to aid in the development
105+
or maintenance of the product, even if at low FTE. This work could include: code development, PR review, consulting activities, etc. As authors are
106+
the most visible names associated with the package, they should be able to offer some type of support if contacted by a user.
107+
108+
🦋 **Acknowledgement (i.e. individual listed in the [README Acknowledgments](https://pharmaverse.github.io/admiral/cran-release/#acknowledgments))**: This
109+
is for users who have made a contribution to the package that does not yet meet the level for authorship as described above. Individuals who were previously
110+
authors but have moved away from the package will also be moved to the Acknowledgements section.
111+
112+
Companies will be listed on the license file as long as either of the following apply:
113+
114+
🦋 They employ the maintainer;
115+
116+
🦋 They employ two or more authors in the development team.
117+
118+
These guidelines should be used both for `{admiral}` and any related packages such as `{admiraldev}`, therapeutic area extension packages and data packages.
119+
120+
__Note: The package maintainer is the final decision-maker as to how contributions are acknowledged, and reserves their discretion to make this choice; there
121+
may be occasions where the guidelines listed above need to be flexed due to the context of the work. For instance, if package is inactive or very
122+
stable, the level of contributions for authorship of acknowledgements may be lower.__

.lycheeignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,3 +8,4 @@ https://rsc.niaid.nih.gov/clinical-research-sites/daids-adverse-event-grading-ta
88
https://rsc.niaid.nih.gov/sites/default/files/daidsgradingcorrectedv21.pdf
99
https://www.fda.gov/media/116737/download
1010
https://documentation.sas.com/doc/en/pgmsascdc/9.4_3.5/movefile/n1xbwdre0giahfn11c99yjkpi2yb.htm
11+
https://www.iso.org/iso-3166-country-codes.html

NEWS.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,9 @@
2323
- A night mode option has been added to the `{admiral}` website. This can be be toggled using the sun/moon icon in the top right corner
2424
of the navigation bar. Tooltips have also been set up for each of the icons on the navigation bar. (#2959)
2525

26+
- The [contributions guidelines](https://pharmaverse.github.io/admiral/cran-release/CONTRIBUTING.html) for the `{admiral}` ecosystem
27+
of packages were updated to specify the level of contributions required for authorship/acknowledgments. (#2979)
28+
2629
## Various
2730

2831
<details>

_pkgdown.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,6 @@ home:
6363
href: articles/templates.html
6464

6565
authors:
66-
before: "We define *authors* as those who are actively maintaining the code base, and *contributors* as those who made a significant contribution in the past. For all acknowledgements, see the eponymous section in the [Home Page](https://pharmaverse.github.io/admiral/cran-release/)."
6766
footer:
6867
roles: [aut, ctb, cre]
6968
text: "Developed by F. Hoffmann-La Roche AG, GlaxoSmithKline LLC,"

inst/WORDLIST

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -281,6 +281,7 @@ Thanikachalam
281281
Thrombocytopenia
282282
Timepoint
283283
Titer
284+
Tooltips
284285
ULN
285286
USCV
286287
USUBJID

0 commit comments

Comments
 (0)