-
Notifications
You must be signed in to change notification settings - Fork 5
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
24 changed files
with
20,574 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,174 @@ | ||
# Contribution Guidelines | ||
|
||
If you're reading this, you're probably thinking about contributing to this | ||
repository. We really appreciate that--thank you! | ||
|
||
This document provides guidelines on contributing to this repository. Please | ||
follow these guidelines when creating issues, making commits, and submitting | ||
pull requests. The repository maintainers review all pull requests and verify | ||
that they conform to these guidelines before approving and merging. | ||
|
||
## Table Of Contents | ||
|
||
- [Contribution Guidelines](#contribution-guidelines) | ||
- [Table Of Contents](#table-of-contents) | ||
- [How Can I Contribute?](#how-can-i-contribute) | ||
- [Contribution Ideas](#contribution-ideas) | ||
- [What should I know before I get started?](#what-should-i-know-before-i-get-started) | ||
- [Guidelines](#guidelines) | ||
- [Licensing](#licensing) | ||
- [Developer’s Certificate of Origin](#developers-certificate-of-origin) | ||
- [Sign Your Work](#sign-your-work) | ||
- [Coding Conventions](#coding-conventions) | ||
- [Additional Notes](#additional-notes) | ||
- [Resources](#resources) | ||
|
||
## How Can I Contribute? | ||
|
||
### Contribution Ideas | ||
|
||
1. Raise issues for bugs, features, and enhancements. | ||
1. Submit updates and improvements to the documentation. | ||
1. Submit articles and guides, which are also part of the documentation. | ||
1. Help out repo maintainers by answering questions in | ||
[Airheads Developer Community][airheads-link]. | ||
1. Share feedback and let us know about interesting use cases in | ||
[Airheads Developer Community][airheads-link]. | ||
|
||
### What should I know before I get started? | ||
|
||
The best way to directly collaborate with the project contributors is through | ||
GitHub. | ||
|
||
- If you want to raise an issue such as a defect, an enhancement request, | ||
feature request, or a general issue, please open a GitHub issue. | ||
- If you want to contribute to our code by either fixing a problem, enhancing | ||
some code, or creating a new feature, please open a GitHub pull request | ||
against the development branch. | ||
|
||
> **Note:** All pull requests require an associated issue number, must be made | ||
> against the **development** branch, and require acknowledgement of the DCO. | ||
> See the [Licensing](#licensing) section below. | ||
Before you start to code, we recommend discussing your plans through a GitHub | ||
issue, especially for more ambitious contributions. This gives other | ||
contributors a chance to point you in the right direction, give you feedback | ||
on your design, and help you find out if someone else is working on the same | ||
thing. | ||
|
||
It is your responsibility to test and verify, prior to submitting a pull | ||
request, that your updated code doesn't introduce any bugs. Please write a | ||
clear commit message for each commit. Brief messages are fine for small | ||
changes, but bigger changes warrant a little more detail (at least a few | ||
sentences). | ||
|
||
Note that all patches from all contributors get reviewed. After a pull request | ||
is made, other contributors will offer feedback. If the patch passes review, a | ||
maintainer will accept it with a comment. When a pull request fails review, the | ||
author is expected to update the pull request to address the issue until it | ||
passes review and the pull request merges successfully. | ||
|
||
At least one review from a maintainer is required for all patches. | ||
|
||
### Guidelines | ||
|
||
This repo is maintained on a best-effort basis. The burden is on the submitter | ||
and not the repo maintainers to ensure the following criteria are met when code | ||
is submitted. | ||
|
||
- All code submissions must adhere to the structure of the repo: | ||
- Lower-level functions and API calls must be saved in the /pyclearpass folder. | ||
- High-level process-focused functions must be saved in the /workflows folder. | ||
- Do not create new separate folders for submitted projects. | ||
- Do not make copies of existing files to be saved in different folders. | ||
- The objective is that all submissions build on the repo as a whole, rather | ||
than creating multiple sub-projects housed in the repo. | ||
- All Python code should conform to PEP-8 standards. The maintainers use | ||
Pycharm IDE to perform this check. That does not require submitters to use | ||
Pycharm, but regardless of the code editor used, the PEP-8 check must be | ||
successful. | ||
- All functions should have explanatory docstrings using the reStructuredText | ||
format. | ||
- All workflows should have a comment at the top explaining the configuration | ||
steps the workflow performs, and any preconditions that need to be met | ||
before running the script. | ||
- Git commits should have clear, concise messages which explain the | ||
changes made in the commit. | ||
- Pull Requests (PRs) should contain a title and comments that explain the | ||
impact of the PR. | ||
- All code submitted for merge consideration must be tested by the submitter. | ||
|
||
## Licensing | ||
|
||
All contributions must include acceptance of the DCO: | ||
|
||
### Developer’s Certificate of Origin | ||
|
||
> Developer Certificate of Origin Version 1.1 | ||
> | ||
> Copyright © 2004, 2006 The Linux Foundation and its contributors. 660 | ||
> York Street, Suite 102, San Francisco, CA 94110 USA | ||
> | ||
> Everyone is permitted to copy and distribute verbatim copies of this | ||
> license document, but changing it is not allowed. | ||
> | ||
> Developer's Certificate of Origin 1.1 | ||
> | ||
> By making a contribution to this project, I certify that: | ||
> | ||
> (a) The contribution was created in whole or in part by me and I have | ||
> the right to submit it under the open source license indicated in the | ||
> file; or | ||
> | ||
> (b) The contribution is based upon previous work that, to the best of my | ||
> knowledge, is covered under an appropriate open source license and I | ||
> have the right under that license to submit that work with | ||
> modifications, whether created in whole or in part by me, under the same | ||
> open source license (unless I am permitted to submit under a different | ||
> license), as indicated in the file; or | ||
> | ||
> (c) The contribution was provided directly to me by some other person | ||
> who certified (a), (b) or (c) and I have not modified it. | ||
> | ||
> (d) I understand and agree that this project and the contribution are | ||
> public and that a record of the contribution (including all personal | ||
> information I submit with it, including my sign-off) is maintained | ||
> indefinitely and may be redistributed consistent with this project or | ||
> the open source license(s) involved. | ||
### Sign Your Work | ||
|
||
To accept the DCO, simply add this line to each commit message with your | ||
name and email address (`git commit -s` will do this for you): | ||
|
||
```text | ||
Signed-off-by: Jane Example <[email protected]> | ||
``` | ||
|
||
For legal reasons, no anonymous or pseudonymous contributions are | ||
accepted. | ||
|
||
## Coding Conventions | ||
|
||
1. Python code should conform to PEP-8. PyCharm editor has a built-in PEP-8 | ||
checker. | ||
1. Since this is a collaborative project, document your code with comments that | ||
will help other contributors understand the code you write. | ||
1. When in doubt, follow conventions you see used in the source already. | ||
|
||
## Additional Notes | ||
|
||
> **Note:** Please don't file an issue to ask a question. Please reach out to | ||
> us via email or disucssion forums. | ||
### Resources | ||
|
||
| Resource | Description | | ||
| --- | --- | | ||
| [Airheads Developer Community][airheads-link] | Aruba Airheads forum to discuss all things network automation. | | ||
| [Aruba DevHub ClearPass Guides][devhub-link]| Aruba DevHub guides for ClearPass | | ||
| [[email protected]][email-link] | Distribution list email to contact the Aruba Automation technical marketing engineering team. | | ||
|
||
[airheads-link]: https://community.arubanetworks.com/t5/Developer-Community/bd-p/DeveloperCommunity | ||
[devhub-link]: https://developer.arubanetworks.com/aruba-cppm/docs | ||
[email-link]: mailto:[email protected] |
File renamed without changes.
Oops, something went wrong.