Skip to content

Commit 9c78e5c

Browse files
maelleyabellini
andauthored
Improve GitHub workflow guidance (#606)
Co-authored-by: Yanina Bellini Saibene <yabellini@gmail.com>
1 parent c1071e5 commit 9c78e5c

File tree

4 files changed

+24
-7
lines changed

4 files changed

+24
-7
lines changed

NEWS.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
https://github.com/ropensci/dev_guide/blob/master/appendix.Rmd#news
1+
https://github.com/ropensci/dev_guide/blob/main/booknews.Rmd

booknews.Rmd

+5-1
Original file line numberDiff line numberDiff line change
@@ -2,13 +2,17 @@
22

33
## dev (0.9.0)
44

5+
- 2022-07-04, Add resources around GitHub workflows (#479, `@maurolepore`).
6+
57
- 2023-02-14, update instructions for CITATION to reflect new CRAN policies (#604, #609)
68

79
- 2023-02-14, add package maintainer cheatsheet (#608).
810

911
- 2023-01-25, add Mastodon as social media (#592, by @yabellini).
1012

11-
- 2023-01-20, fix small formating error (#590 by @eliocamp).
13+
- 2023-01-25, add Mastodon as social media (#592, by `@yabellini`).
14+
15+
- 2023-01-20, fix small formating error (#590 by `@eliocamp`).
1216

1317
- 2022-11-22, mention shinytest2 near shinytest.
1418

maintenance_collaboration.Rmd

+10-4
Original file line numberDiff line numberDiff line change
@@ -105,20 +105,26 @@ A possible model for onboarding collaborators is provided by Jim Hester in [his
105105

106106
If your problem is _recruiting_ collaborators, you can post an open call like Jim Hester's [on Twitter](https://twitter.com/jimhester_/status/997109466674819074), [GitHub](https://github.com/jimhester/lintr/issues/318), and as an rOpenSci package author, you can ask for help in rOpenSci slack and ask rOpenSci team for ideas for recruiting new collaborators.
107107

108-
### Working with collaborators (including yourself) {#gitflow}
108+
### Working with collaborators (including yourself) {#git-workflow}
109109

110110
[Branches](https://happygitwithr.com/git-branches.html) are cheap. Use them extensively when developing features, testing out new ideas, fixing problems.
111111

112112
One of the branches is the default / main branch, where, if you follow [trunk-based development](https://www.atlassian.com/continuous-delivery/continuous-integration/trunk-based-development), you "merge small, frequent updates".
113113
See also [GitHub flow](https://docs.github.com/en/get-started/quickstart/github-flow) and [GitLab flow](https://docs.gitlab.com/ee/topics/gitlab_flow.html) docs.
114-
You might want to pair the frequent incrementing of version numbers (in `DESCRIPTION`).
114+
You might want to frequently increment version numbers (in `DESCRIPTION`).
115115
One particular aspect of working with collaborators is reviewing pull requests, with some useful guidance in:
116116

117-
* [The Art of Giving and Receiving Code Reviews (Gracefully), by Alex Hill](https://www.alexandra-hill.com/2018/06/25/the-art-of-giving-and-receiving-code-reviews/)
118-
* [GitHub documentation about PR reviews](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/about-pull-request-reviews)
117+
* [The Art of Giving and Receiving Code Reviews (Gracefully), by Alex Hill](https://www.alexandra-hill.com/2018/06/25/the-art-of-giving-and-receiving-code-reviews/);
118+
* [GitHub documentation about PR reviews](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/about-pull-request-reviews).
119+
120+
You might want to tinker with your GitHub repository settings to, for instance, [require pull request reviews before merging](https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/defining-the-mergeability-of-pull-requests/about-protected-branches#require-pull-request-reviews-before-merging=).
121+
See also GitHub docs about ["code owners"](https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners).
119122

120123
For making and reviewing pull requests we recommend [exploring usethis functions](https://usethis.r-lib.org/articles/pr-functions.html).
121124

125+
For your "git remote" setup refer to [happy git with r](https://happygitwithr.com/common-remote-setups.html).
126+
See also [Useful Git patterns for real life](https://happygitwithr.com/workflows-intro.html) in the same book.
127+
122128
### Be generous with attributions {#attributions}
123129

124130
If someone contributes to your repository consider adding them in DESCRIPTION, as contributor ("ctb") for small contributions, author ("aut") for bigger contributions. Traditionally when citing a package in a scientific publication only "aut" authors are listed, not "ctb" contributors; and on `pkgdown` websites only "aut" names are listed on the homepage, all authors being listed on the authors/ page.

maintenance_collaboration.es.Rmd

+8-1
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ Jim Hester ofrece un posible modelo de incorporación de miembros de equipo en [
110110

111111
Si tu problema es *reclutar* nuevas personas, puedes publicar una convocatoria abierta como la de Jim Hester [en Twitter](https://twitter.com/jimhester_/status/997109466674819074), [GitHub](https://github.com/jimhester/lintr/issues/318) y, como responsable de un paquete de rOpenSci, puedes pedir ayuda en el Slack de rOpenSci y solicitar al equipo de rOpenSci ideas para reclutar ayuda.
112112

113-
### Trabajar con otras personas (y incluyendo tú en el futuro) {#gitflow}
113+
### Trabajar con otras personas (incluyendo tú en el futuro) {#git-workflow}
114114

115115
[Las *branches*](https://happygitwithr.com/git-branches.html) son baratas.
116116
Utilízalas mucho cuando desarrolles funciones, pruebes nuevas ideas o arregles problemas.
@@ -124,8 +124,15 @@ Pueden ver algunas guías útiles en:
124124
- [*The Art of Giving and Receiving Code Reviews (Gracefully)* (El arte de dar y recibir revisiones de código (con elegancia)), por Alex Hill](https://www.alexandra-hill.com/2018/06/25/the-art-of-giving-and-receiving-code-reviews/)
125125
- [Documentación de GitHub sobre revisiones de PR](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/about-pull-request-reviews)
126126

127+
Puedes modificar la configuración de tu repositorio de GitHub,
128+
por ejemplo [requeriendo revisiones de solicitudes de cambio antes de aceptarlas e incorporarlas](https://docs.github.com/es/repositories/configuring-branches-and-merges-in-your-repository/defining-the-mergeability-of-pull-requests/about-protected-branches#require-pull-request-reviews-before-merging).
129+
También puedes consultar la documentación [acerca de propiedad sobre código](https://docs.github.com/es/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners).
130+
127131
Para hacer y revisar *pull requests* recomendamos [explorar las funciones del paquete usethis](https://usethis.r-lib.org/articles/pr-functions.html).
128132

133+
Para tu configuración "git remote" consulta [_Happy Git with R_](https://happygitwithr.com/common-remote-setups.html).
134+
También es útil la sección [Useful Git patterns for real life](https://happygitwithr.com/workflows-intro.html) en el mismo libro.
135+
129136
### Atribuye con generosidad {#attributions}
130137

131138
Si una persona contribuye a tu repositorio, considera añadirla en el archivo DESCRIPTION, con el rol de contribución ("ctb") para pequeñas contribuciones o autoría ("aut") para contribuciones mayores.

0 commit comments

Comments
 (0)