forked from sonic-net/sonic-swss
-
Notifications
You must be signed in to change notification settings - Fork 2
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
0 parents
commit e6727bc
Showing
1 changed file
with
42 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,42 @@ | ||
# SONiC: Software for Open Networking in the Cloud | ||
|
||
## sonic-platform-common | ||
|
||
Collection of Python packages which provide a common interface to platform-specific hardware peripherals in SONiC | ||
|
||
# Contribution guide | ||
|
||
All contributors must sign a contribution license agreement (CLA) before contributions can be accepted. This process is now automated via a GitHub bot when submitting new pull request. If the contributor has not yet signed a CLA, the bot will create a comment on the pull request containing a link to electronically sign the CLA. | ||
|
||
### GitHub Workflow | ||
|
||
We're following basic GitHub Flow. If you have no idea what we're talking about, check out [GitHub's official guide](https://guides.github.com/introduction/flow/). Note that merge is only performed by the repository maintainer. | ||
|
||
Guide for performing commits: | ||
|
||
* Isolate each commit to one component/bugfix/issue/feature | ||
* Use a standard commit message format: | ||
|
||
> [component/folder touched]: Description intent of your changes | ||
> | ||
> [List of changes] | ||
> | ||
> Signed-off-by: Your Name [email protected] | ||
For example: | ||
|
||
> swss-common: Stabilize the ConsumerTable | ||
> | ||
> * Fixing autoreconf | ||
> * Fixing unit-tests by adding checkers and initialize the DB before start | ||
> * Adding the ability to select from multiple channels | ||
> * Health-Monitor - The idea of the patch is that if something went wrong with the notification channel, | ||
> we will have the option to know about it (Query the LLEN table length). | ||
> | ||
> Signed-off-by: John Doe [email protected] | ||
|
||
* Each developer should fork this repository and [add the team as a Contributor](https://help.github.com/articles/adding-collaborators-to-a-personal-repository) | ||
* Push your changes to your private fork and do "pull-request" to this repository | ||
* Use a pull request to do code review | ||
* Use issues to keep track of what is going on |