|
1 | 1 | # Contributing |
2 | 2 |
|
3 | | -We welcome contributions to the extendr project. Contributions come in many forms. Please carefully read and follow these guidelines. This will help us make the contribution process easy and effective for everyone involved. It also communicates that you agree to respect the time of the developers managing and developing this project. |
| 3 | +Contributions to the extendr project can come in many forms, and we welcome all |
| 4 | +of them. Please carefully read and follow the guidelines in our |
| 5 | +[Contributing Guide](https://extendr.github.io/contributing/index.html). This |
| 6 | +will help us make the contribution process easy and effective for everyone |
| 7 | +involved. It also communicates that you agree to respect the time of the |
| 8 | +developers managing and developing this project. |
4 | 9 |
|
| 10 | +## Code of Conduct {#code-of-conduct} |
5 | 11 |
|
6 | | -## Quicklinks |
| 12 | +We take our open source community seriously and hold ourselves and other |
| 13 | +contributors to high standards of communication. By participating and |
| 14 | +contributing to this project, you agree to uphold our [Code of |
| 15 | +Conduct.](https://github.com/extendr/extendr/blob/master/CODE-OF-CONDUCT.md) |
7 | 16 |
|
8 | | -* [Code of Conduct](#code-of-conduct) |
9 | | -* [Getting Started](#getting-started) |
10 | | - * [Issues](#issues) |
11 | | - * [Pull Requests](#pull-requests) |
12 | | -* [Getting Help](#getting-help) |
13 | | -* [Authorship](#authorship) |
14 | | -* [Attribution](#attribution) |
| 17 | +## Getting Started {#getting-started} |
15 | 18 |
|
16 | | -## Repo structure |
17 | | - |
18 | | -This is the source of the extendr project's website. |
19 | | - |
20 | | -- `user-guide` Is the user guide for using `extendr-api` |
21 | | -- `blog` is the source code for blogs for the extendr-project |
22 | | -- `intro-rust` contains souce code for an Introduction to Rust for R developers |
23 | | - |
24 | | -## Using extendr code chunks |
25 | | - |
26 | | -Whenever using extendr code, use the `extendrsrc` and `extendr` knitr engines. |
27 | | -The user-guide/serde-integration.qmd is the best example. Try to emulate this style. |
28 | | - |
29 | | -## User guide |
30 | | - |
31 | | -The user guide is a friendly introduction to the `extendr-api` crate. Any entries into the user guide should be **concise** and use **simple language**. Always use code chunks whenever using extendr code. |
32 | | - |
33 | | -The structure is roughly as follows: |
34 | | - |
35 | | -```` |
36 | | -``` |
37 | | ---- |
38 | | -title: "An informative title" |
39 | | ---- |
40 | | -
|
41 | | -- Motivate the functionality that the entry will cover |
42 | | -- No more than 4 sentences |
43 | | -
|
44 | | -## The feature |
45 | | -
|
46 | | -- Describe the functionality that will be covered |
47 | | -- Discuss pre-reqs if any |
48 | | -
|
49 | | -## Basic usage |
50 | | -
|
51 | | -- Provide a basic example |
52 | | -- Discuss your code prior to showing the code |
53 | | -
|
54 | | -```{extendrsrc} |
55 | | -// your rust code |
56 | | -``` |
57 | | -
|
58 | | -- Summarise and explicate |
59 | | -
|
60 | | -## Advanced usage |
61 | | -
|
62 | | -- Motivate advanced usage |
63 | | -- Provide an example |
64 | | -
|
65 | | -```{extendrsrc} |
66 | | -// your rust code |
67 | | -``` |
68 | | -
|
69 | | -- Summarize and explicate |
70 | | -
|
71 | | -## See also |
72 | | -
|
73 | | -- Provide useful links to other docs on the site or externally |
74 | | -``` |
75 | | -```` |
76 | | - |
77 | | -### Writing style |
78 | | - |
79 | | -- use of second person is fine, but you cannot write about one's intention in the second person. |
80 | | -- Banned phrases ❌: |
81 | | - - "you want to" |
82 | | - - "you need to" |
83 | | -- never use `print()` to print an R object |
84 | | - |
85 | | - |
86 | | -## Code of Conduct |
87 | | - |
88 | | -We take our open source community seriously and hold ourselves and other contributors to high standards of communication. By participating and contributing to this project, you agree to uphold our [Code of Conduct.](https://github.com/extendr/extendr/blob/master/CODE-OF-CONDUCT.md) |
89 | | - |
90 | | -## Getting Started |
91 | | - |
92 | | -Contributions can be made via Issues and Pull Requests (PRs). A few general guidelines cover both: |
| 19 | +Contributions can be made via Issues and Pull Requests (PRs). A few general |
| 20 | +guidelines cover both: |
93 | 21 |
|
94 | 22 | - Please search for existing Issues and PRs before creating your own. |
95 | | -- We work hard to makes sure issues are handled in a timely manner but, depending on the problem and maintainer availability, it could take a while to investigate the problem. A friendly ping in the comment thread can help draw attention if an issue has not received any attention for a while. Please keep in mind that all contributors to this project are volunteers and may have other commitments they need to attend to. |
| 23 | +- We work hard to makes sure issues are handled in a timely manner but, |
| 24 | + depending on the problem and maintainer availability, it could take a while |
| 25 | + to investigate the problem. A friendly ping in the comment thread can help |
| 26 | + draw attention if an issue has not received any attention for a while. |
| 27 | + Please keep in mind that all contributors to this project are volunteers and |
| 28 | + may have other commitments they need to attend to. |
96 | 29 |
|
97 | | -### Issues |
| 30 | +### Issues {#issues} |
98 | 31 |
|
99 | | -Issues should be used to report problems with the library, request a new feature, or to discuss potential changes before a PR is created. Please **do not** use Issues to request user support. |
| 32 | +Issues should be used to report problems with the library, request a new |
| 33 | +feature, or to discuss potential changes before a PR is created. Please **do |
| 34 | +not** use Issues to request user support. |
100 | 35 |
|
101 | | -Whenever possible, please provide a minimal reproducible example (reprex) to any bug report that you are filing. The more minimal your example, the more likely that somebody else can figure out what the problem is, so please remove any code that isn't relevant to the problem you are reporting. |
| 36 | +Whenever possible, please provide a minimal reproducible example (reprex) to any |
| 37 | +bug report that you are filing. The more minimal your example, the more likely |
| 38 | +that somebody else can figure out what the problem is, so please remove any code |
| 39 | +that isn't relevant to the problem you are reporting. |
102 | 40 |
|
103 | | -Please keep issues focused on one particular problem. Don't feel shy about opening multiple issues if you're encountering more than one problem. |
| 41 | +Please keep issues focused on one particular problem. Don't feel shy about |
| 42 | +opening multiple issues if you're encountering more than one problem. |
104 | 43 |
|
105 | | -If you find an Issue that addresses the problem you're having, please add your own reproduction information to the existing issue rather than creating a new one. Adding a [reaction](https://github.blog/2016-03-10-add-reactions-to-pull-requests-issues-and-comments/) can also help be indicating to our maintainers that a particular problem is affecting more than just the reporter. |
| 44 | +If you find an Issue that addresses the problem you're having, please add your |
| 45 | +own reproduction information to the existing issue rather than creating a new |
| 46 | +one. Adding a |
| 47 | +[reaction](https://github.blog/2016-03-10-add-reactions-to-pull-requests-issues-and-comments/) |
| 48 | +can also help indicate to our maintainers that a particular problem is affecting |
| 49 | +more than just the reporter. |
106 | 50 |
|
107 | | -### Pull Requests |
| 51 | +### Pull Requests {#pull-requests} |
108 | 52 |
|
109 | | -PRs are always welcome and can be a quick way to get your fix or improvement slated for the next release. However, please always open an Issue before submitting a PR. |
| 53 | +PRs are always welcome and can be a quick way to get your fix or improvement |
| 54 | +slated for the next release. However, please always open an Issue before |
| 55 | +submitting a PR. |
110 | 56 |
|
111 | 57 | In general, PRs should: |
112 | 58 |
|
113 | 59 | - Address a single concern in the least number of changed lines as possible. |
114 | | -- Only fix/add the functionality in question **OR** address wide-spread whitespace/style issues, not both. |
| 60 | +- Only fix/add the functionality in question **OR** address wide-spread |
| 61 | + whitespace/style issues, not both. |
115 | 62 | - Add unit or integration tests for fixed or changed functionality. |
116 | 63 | - Include documentation. |
117 | | -- Indicate which Issue they address by using the words `Closes #<issue number>` or `Fixes #<issue number>` in the body of the PR and/or the git commit message. (See the [GitHub Documentation](https://docs.github.com/en/free-pro-team@latest/github/managing-your-work-on-github/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword) for details about linking PRs to Issues and automatically closing Issues when merging PRs.) |
118 | | - |
| 64 | +- Indicate which Issue they address by using the words `Closes #<issue number>` |
| 65 | + or `Fixes #<issue number>` in the body of the PR and/or the git commit |
| 66 | + message. (See the [GitHub Documentation](https://docs.github.com/en/free-pro-team@latest/github/managing-your-work-on-github/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword) |
| 67 | + for details about linking PRs to Issues and automatically closing Issues when |
| 68 | + merging PRs.) |
119 | 69 |
|
120 | | -In general, we follow the [GitHub flow](https://guides.github.com/introduction/flow/index.html) development model: |
| 70 | +In general, we follow the [GitHub flow](https://guides.github.com/introduction/flow/index.html) |
| 71 | +development model: |
121 | 72 |
|
122 | 73 | 1. Fork the repository to your own Github account |
123 | 74 | 2. Clone the project to your machine |
124 | 75 | 3. Create a branch locally with a succinct but descriptive name |
125 | 76 | 4. Commit changes to the branch |
126 | 77 | 5. Push changes to your fork |
127 | | -6. Open a PR in our repository and follow the PR template so that we can efficiently review the changes. |
| 78 | +6. Open a PR in our repository and follow the PR template so that we can |
| 79 | + efficiently review the changes. |
128 | 80 |
|
129 | | -## Getting Help |
| 81 | +## Getting Help {#getting-help} |
130 | 82 |
|
131 | | -Please join us on our [Discord server](https://discord.gg/7hmApuc) for general conversations and questions that don't belong into a GitHub issue. |
| 83 | +Please join us on our [Discord server](https://discord.gg/7hmApuc) for general |
| 84 | +conversations and questions that don't belong in a GitHub issue. |
132 | 85 |
|
133 | | -## Authorship |
| 86 | +## Authorship {#authorship} |
134 | 87 |
|
135 | | -Contributors who have made multiple, sustained, and/or non-trivial contributions to the project may be added to the author list. New author names will always be added at the end of the list, so that author order reflects chronological order of joining the project. All authorship decisions are at the discretion of the current maintainers of the project. |
| 88 | +Contributors who have made multiple, sustained, and/or non-trivial contributions |
| 89 | +to the project may be added to the author list. New author names will always be |
| 90 | +added at the end of the list, so that author order reflects chronological order |
| 91 | +of joining the project. All authorship decisions are at the discretion of the |
| 92 | +current maintainers of the project. |
136 | 93 |
|
137 | | -## Attribution |
| 94 | +## Attribution {#attribution} |
138 | 95 |
|
139 | | -This document was adapted from the [General Contributing Guidelines](https://github.com/auth0/open-source-template/blob/master/GENERAL-CONTRIBUTING.md) of the auth0 project. |
| 96 | +This document was adapted from the [General Contributing Guidelines](https://github.com/auth0/open-source-template/blob/master/GENERAL-CONTRIBUTING.md) |
| 97 | +of the auth0 project. |
0 commit comments