Skip to content

Commit

Permalink
(WIP) Add Contributor Guide
Browse files Browse the repository at this point in the history
  • Loading branch information
per1234 committed Nov 9, 2023
1 parent 4304923 commit aa33dea
Show file tree
Hide file tree
Showing 11 changed files with 459 additions and 1 deletion.
45 changes: 45 additions & 0 deletions .github/ISSUE_TEMPLATE/bug-report.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,45 @@
# Source: https://github.com/arduino/tooling-project-assets/blob/main/issue-templates/forms/general/bug-report.yml
# See: https://docs.github.com/communities/using-templates-to-encourage-useful-issues-and-pull-requests/syntax-for-issue-forms

name: Bug report
description: Report a problem with the code or documentation in this repository.
labels:
- "type: imperfection"
body:
- type: textarea
id: description
attributes:
label: Describe the problem
validations:
required: true
- type: textarea
id: reproduce
attributes:
label: To reproduce
description: Provide the specific set of steps we can follow to reproduce the problem.
validations:
required: true
- type: textarea
id: expected
attributes:
label: Expected behavior
description: What would you expect to happen after following those instructions?
validations:
required: true
- type: textarea
id: additional
attributes:
label: Additional context
description: Add any additional information here.
validations:
required: false
- type: checkboxes
id: checklist
attributes:
label: Issue checklist
description: Please double-check that you have done each of the following things before submitting the issue.
options:
- label: I searched for previous reports in [the issue tracker](https://github.com/per1234/inoplatforms/issues?q=)
required: true
- label: My report contains all necessary details
required: true
21 changes: 21 additions & 0 deletions .github/ISSUE_TEMPLATE/config.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
# Source:
# https://github.com/arduino/tooling-project-assets/blob/main/issue-templates/template-choosers/general/config.yml
# See:
# https://docs.github.com/communities/using-templates-to-encourage-useful-issues-and-pull-requests/configuring-issue-templates-for-your-repository#configuring-the-template-chooser

contact_links:
- name: Learn about this project
url: https://github.com/per1234/inoplatforms#readme
about: Detailed documentation is available here.
- name: Learn about the **inoplatforms** catalog
url: https://inoplatforms.info/about/
about: Detailed documentation is available here.
- name: Support request
url: https://forum.arduino.cc/
about: We can help you out on the Arduino Forum!
- name: Issue report guide
url: https://inoplatforms.info/contributor-guide/issues/
about: Learn about submitting issue reports to this repository.
- name: Contributor guide
url: https://inoplatforms.info/contributor-guide/
about: Learn about contributing to this project.
40 changes: 40 additions & 0 deletions .github/ISSUE_TEMPLATE/feature-request.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,40 @@
# Source: https://github.com/arduino/tooling-project-assets/blob/main/issue-templates/forms/general/bug-report.yml
# See: https://docs.github.com/communities/using-templates-to-encourage-useful-issues-and-pull-requests/syntax-for-issue-forms

name: Feature request
description: Suggest an enhancement to this project.
labels:
- "type: enhancement"
body:
- type: textarea
id: description
attributes:
label: Describe the request
validations:
required: true
- type: textarea
id: current
attributes:
label: Describe the current behavior
description: |
What is the current behavior of the project in relation to your request?
How can we reproduce that behavior?
validations:
required: true
- type: textarea
id: additional
attributes:
label: Additional context
description: Add any additional information here.
validations:
required: false
- type: checkboxes
id: checklist
attributes:
label: Issue checklist
description: Please double-check that you have done each of the following things before submitting the issue.
options:
- label: I searched for previous requests in [the issue tracker](https://github.com/per1234/inoplatforms/issues?q=)
required: true
- label: My request contains all necessary details
required: true
4 changes: 3 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,9 @@ The list can be viewed online [here](ino-hardware-package-list.tsv).

### Contributing

Additions/corrections/updates to the list are welcome! Please submit a pull request or issue.
Additions/corrections/updates to the list are welcome!

See [the **Contributor Guide**](https://www.inoplatforms.info/contributor-guide/).

### Acknowledgments

Expand Down
21 changes: 21 additions & 0 deletions docs/CONTRIBUTING.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
<!-- Source: https://github.com/arduino/tooling-project-assets/blob/main/documentation-templates/contributor-guide/general/CONTRIBUTING.md -->

# Contributor Guide

Thanks for your interest in contributing to this project!

There are several ways you can get involved:

| Type of contribution | Contribution method |
| ----------------------------------------- | ----------------------------------------------- |
| - Support<br/>- Question<br/>- Discussion | Post on the [**Arduino Forum**][forum] |
| - Bug report<br/>- Feature request | Issue report (see the guide [**here**][issues]) |
| - Bug fix<br/>- Enhancement | Pull request (see the guide [**here**][prs]) |

[forum]: https://forum.arduino.cc
[issues]: https://inoplatforms.info/contributor-guide/issues/
[prs]: https://inoplatforms.info/contributor-guide/pull-requests/

## Resources

- [**Development Guide**](https://inoplatforms.info/development-guide/)
25 changes: 25 additions & 0 deletions site/content/contributor-guide/_index.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
---
title: Contributor Guide
breadcrumb: contribute
layout: basic
---

<!-- Source: https://github.com/arduino/tooling-project-assets/blob/main/documentation-templates/contributor-guide/general/CONTRIBUTING.md -->

Thanks for your interest in contributing to this project!

There are several ways you can get involved:

| Type of contribution | Contribution method |
| ----------------------------------------- | ----------------------------------------------- |
| - Support<br/>- Question<br/>- Discussion | Post on the [**Arduino Forum**][forum] |
| - Bug report<br/>- Feature request | Issue report (see the guide [**here**][issues]) |
| - Bug fix<br/>- Enhancement | Pull request (see the guide [**here**][prs]) |

[forum]: https://forum.arduino.cc
[issues]: /contributor-guide/issues/
[prs]: /contributor-guide/pull-requests/

## Resources

- [**Development Guide**](/development-guide/)
35 changes: 35 additions & 0 deletions site/content/contributor-guide/issues/_index.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
---
title: Issue Report Guide
breadcrumb: issues
layout: basic
---

<!-- Source: https://github.com/arduino/tooling-project-assets/blob/main/documentation-templates/contributor-guide/general/contributor-guide/issues.md -->

---

❗ Do you need help or have a question about using this project? Support requests should be made to the [Arduino Forum](https://forum.arduino.cc).

---

High quality bug reports and feature requests are valuable contributions to this project. These can be made by submitting an issue report to the project's GitHub repository:

https://github.com/per1234/inoplatforms/issues/new/choose

## Before Reporting an Issue

- Search [existing pull requests and issues](https://github.com/per1234/inoplatforms/issues?q=) to see if it was already reported.<br />
If you have additional information to provide about an existing issue, please comment there instead of creating a duplicate. You can use [GitHub's "Reactions" feature](https://github.blog/2016-03-10-add-reactions-to-pull-requests-issues-and-comments/) if you only want to express support 👍.

## Qualities of an Excellent Report

- Concise and descriptive issue title.<br />
Vague titles make it difficult to decipher the purpose of the issue when looking through the list of reports, which might result in your issue not being given proper attention.
- Describe the issue and what behavior you were expecting.<br />
Include the full and exact text of any relevant error or warning messages you might have encountered.
- Provide a full set of steps necessary to reproduce the issue.<br />
Demonstration code or commands should be complete and simplified to the minimum necessary to reproduce the issue.
- Be responsive.<br />
We may need you to provide additional information in order to investigate and resolve the issue.<br />
Make sure your GitHub account is configured so that you will receive notifications of responses to your issue report.
- If you find a solution to your problem, please comment on your issue report with an explanation of how you were able to fix it, then close the issue.
191 changes: 191 additions & 0 deletions site/content/contributor-guide/pull-requests/_index.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,191 @@
---
title: Pull Request Guide
breadcrumb: PRs
layout: basic
---

<!-- Source: https://github.com/arduino/tooling-project-assets/blob/main/documentation-templates/contributor-guide/general/contributor-guide/pull-requests.md -->

A [**pull request**](https://docs.github.com/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/about-pull-requests) (PR) is the mechanism used to propose changes to the content of this project's repository.

If you are looking for ideas of what to work on, check [the list of open issue reports](https://github.com/per1234/inoplatforms/issues). Pull requests addressing any of those bug reports and feature requests are welcome.

## Contribution Workflow

Each contribution travels through a formal process which allows it to be efficiently incorporated into the project.

### 1. Plan

#### Research

Start by searching the repository for existing pull requests and issues related to your planned contribution so you can see any related conversations and proposals and avoid duplicate effort:

https://github.com/per1234/inoplatforms/issues?q=

#### Discussion

It can sometimes be useful to get feedback from others during the planning process.

The Arduino Forum is a good place to discuss any Arduino-related subject with the members of the community.re are a couple good options for discussing planned development work:

https://forum.arduino.cc/

### 2. Fork

Forking a GitHub repository creates a copy of it under your account. You will stage contributions in your fork of this project.

[More information about forking repositories](https://docs.github.com/get-started/quickstart/fork-a-repo)

#### Enabling CI in Your Fork

The repository is configured to run automated [continuous integration](https://wikipedia.org/wiki/Continuous_integration) (CI) checks and tests. It's a good idea to enable CI in your fork so you can make sure your work will pass the checks before you submit a pull request:

1. Open the homepage of your fork in the browser.
1. Click the "**Actions**" tab.
1. Click the <kbd>**I understand my workflows, go ahead and enable them**</kbd> button.
1. Some of the workflows will now need to be activated individually. Perform the following steps for each of the useful workflows listed on the left side of the page that have a "**!**" icon:
1. Click on the workflow name.
1. Click the <kbd>**Enable workflow**</kbd> button.

### 3. Clone

Cloning a repository creates a copy of it on your computer.

It is possible to make simple changes to your repository using the GitHub web interface without cloning the repository. However, the GitHub web interface is quite limiting so you will likely find the need to work with a clone (using **Git** directly or your choice of [Git client software](https://git-scm.com/downloads/guis)) for any significant development work.

[More information about cloning repositories](https://git-scm.com/docs/git-clone)

### 4. Branch

Create a branch in your fork to contain the changes for your contribution. You must make a separate branch in your fork for each pull request you submit.

[More information about branches](https://docs.github.com/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/about-branches)

### 5. Make a change

Some things to keep in mind:

- Make sure your change complies with the project's established style conventions.
- Remember to also update the documentation content in the repository if required by your changes.
- If the project contains a test suite, update or add tests according to your change as appropriate.

See [the development guide](/development-guide/) for more information.

### 6. Test

Test your change carefully to make sure it works correctly and did not break other components of the project.

As a supplement for general testing, the project is set up with automated checks and tests to facilitate development.

See [the development guide](/development-guide/) for instructions.

### 7. Commit

Once the work on your change is complete, add it to the revision history of the Git repository by making a commit.

Make sure to follow the [Commit Guidelines](#commit-guidelines).

[More information about commits](https://git-scm.com/docs/git-commit)

### 8. Push

If you're working from a [clone](#3-clone), you will need to push your commit to your fork on GitHub.

[More information about pushing commits](https://git-scm.com/docs/git-push)

#### Checking CI Results

If you have [enabled CI in your repository](#enabling-ci-in-your-fork), GitHub will run the relevant checks automatically every time you push a commit to your fork.

You can see the results of these checks by doing either of the following:

- Clicking the status icon (✔️ or ❌) shown to the right of a commit.
- Opening the repository's "**Actions**" tab.

### 9. Pull request

A pull request (PR) is a proposal to make a change in a repository. The repository maintainer is able to accept the changes you propose in a pull request by simply clicking a button.

[More information about pull requests](https://docs.github.com/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/about-pull-requests)

#### Scope

Each pull request should address a single bug fix or enhancement. If you have multiple unrelated fixes or enhancements to contribute, submit them as separate pull requests.

#### Description

Pull request title and description should follow [the same guidelines as commit messages](#commit-message).

If your pull request fixes an issue in the issue tracker, use [a closing keyword](https://docs.github.com/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword) in the body to indicate this.

In some cases, it might make sense to request feedback on a proposal before it is ready to be merged. You can indicate this by starting the pull request title with **[WIP]** (work in progress). Once the pull request is ready to be merged, edit the title and remove the "[WIP]".

### 10. Resolve CI failures

Relevant checks will run automatically once you have submitted the pull request. Once these checks are finished, you can see a summary of the results near the bottom of the pull request page:

![checks](assets/checks.png)

Failed checks will be indicated with an ❌. If any checks failed, please fix whatever caused it to fail. Click the "**Details**" link to the right of the check name to open the logs, which provide details about the failure.

---

**** In some rare cases, a CI failure may be unrelated to the changes made in your pull request. So if the information in the logs doesn't seem relevant, please comment on the pull request to ask a maintainer to take a look.

---

When you push to the branch of your fork the pull request was submitted from, the commit is automatically added to the pull request. Don't create a new pull request to fix problems; update the existing pull request.

### 11. Resolve changes requested from reviews

Interested parties may review your pull request and suggest improvements.

To act on general review suggestions, you can add commits to the branch you submitted the pull request from, which will automatically be added to the pull request. Don't create a new pull request to act on review suggestions; update the existing pull request.

Reviewers may suggest specific changes, which can be applied by [clicking the <kbd>**Commit suggestion**</kbd> button](https://docs.github.com/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/incorporating-feedback-in-your-pull-request#applying-suggested-changes).

[More information about pull request reviews](https://docs.github.com/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/about-pull-request-reviews)

### 12. Merge

One of the repository maintainers can now choose to accept your proposed change. Once the pull request is [merged](https://docs.github.com/pull-requests/collaborating-with-pull-requests/incorporating-changes-from-a-pull-request/merging-a-pull-request), you can delete the branch you created in your fork for the pull request and delete the fork as well if you like.

Thanks so much for your contribution!

---

It is possible that the maintainers may decide a pull request doesn't align with Arduino's goals for the project and close it rather than merging. A record of the proposed changes will always be available on GitHub for future reference. If you think your modifications will be of use to others, you are welcome to maintain your own fork of the repository.

---

## Commit Guidelines

The commit history of a repository is an important resource for developers. Repositories may accumulate thousands of commits over the course of decades. Each individual commit contributes either to the commit history being pleasant and efficient to work with, or to it being a confusing mess. For this reason, it's essential for contributors to create clean, high quality commits.

### Scope

Commits must be "atomic". This means that the commit completely accomplishes a single task. Each commit should result in fully functional code. Multiple tasks should not be combined in a single commit, but a single task should not be split over multiple commits (e.g., one commit per file modified is not a good practice).

[More information about atomic commits](https://www.freshconsulting.com/insights/blog/atomic-commits/)

### Commit Message

The commit message documents what the change was and why it was done. A little effort now writing a good commit message can save future developers from wasted time and frustration trying to understand the purpose of a poorly documented commit.

#### Commit Message Title

- Use the [imperative mood](https://cbea.ms/git-commit/#imperative) in the title.<br />
For example:
> Use LED_BUILTIN macro in LED pin definition
- Capitalize the title.
- Do not end the title with punctuation.
- Do not use GitHub's default commit titles (e.g., "Update examples/Foo/Foo.ino").

#### Commit Message Body

- Separate title from the body with a blank line. If you're committing via GitHub or [GitHub Desktop](https://desktop.github.com/) this will be done automatically.
- Wrap body at 120 characters.
- Completely explain the purpose of the commit.<br />
Include a rationale for the change, any caveats, side-effects, etc.

[More information on commit messages](https://cbea.ms/git-commit/)
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading

0 comments on commit aa33dea

Please sign in to comment.