You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
- Support for **pull request from forked repositories** by making use of the new [pull_request_target](https://docs.github.com/en/actions/reference/events-that-trigger-workflows#pull_request_target) event - [Issue#14](https://github.com/cla-assistant/github-action/issues/14)
7
+
- support for storing signatures in a remote repository (including private repository) [Issue#12](https://github.com/cla-assistant/github-action/issues/12)
8
+
- Added support for custom commit and CLA bot pull request comment messages: [PullRequest#33](https://github.com/cla-assistant/github-action/pull/33), Credits: [AnandChowdhary](https://github.com/AnandChowdhary)
9
+
10
+
**Improvements:**
11
+
- complete refactoring of all the files to make the bot more efficient and more readable
12
+
- updated content in README file
13
+
- Changed from using Whitelist to allow list
14
+
15
+
**Fixed Bugs:**
16
+
- CLA check not updated to success when all the contributors have the signed the CLA [Issue#39](https://github.com/cla-assistant/github-action/issues/39)
Copy file name to clipboardExpand all lines: CONTRIBUTING.md
+7-22Lines changed: 7 additions & 22 deletions
Display the source diff
Display the rich diff
Original file line number
Diff line number
Diff line change
@@ -1,6 +1,6 @@
1
-
# Contributing to CLA Assistant GitHub Action
1
+
# Contributing to CLA Assistant
2
2
3
-
You want to contribute to CLA Assistant GitHub Action? Welcome! Please read this document to understand what you can do:
3
+
You want to contribute to CLA Assistant? Welcome! Please read this document to understand what you can do:
4
4
*[Help Others](#help-others)
5
5
*[Analyze Issues](#analyze-issues)
6
6
*[Report an Issue](#report-an-issue)
@@ -81,28 +81,13 @@ You are welcome to contribute code to CLA Assistant in order to fix bugs or to i
81
81
82
82
There are three important things to know:
83
83
84
-
1. You must be aware of the Apache License (which describes contributions) and **agree to the Contributors License Agreement**. This is common practice in all major Open Source projects.
85
-
For company contributors special rules apply. See the respective section below for details.
84
+
1. You must be aware that you need to submit [Developer Certificate of Origin](https://developercertificate.org/) in order for your contribution to be accepted. This is common practice in all major Open Source projects.
86
85
2. There are **several requirements regarding code style, quality, and product standards** which need to be met (we also have to follow them). The respective section below gives more details on the coding guidelines.
87
86
3.**Not all proposed contributions can be accepted**. Some features may e.g. just fit a third-party add-on better. The code must fit the overall direction of CLA Assistant and really improve it. The more effort you invest, the better you should clarify in advance whether the contribution fits: the best way would be to just open an issue to discuss the feature you plan to implement (make it clear you intend to contribute).
88
87
89
-
### Contributor License Agreement
90
-
91
-
When you contribute (code, documentation, or anything else), you have to be aware that your contribution is covered by the same [Apache 2.0 License](https://www.apache.org/licenses/LICENSE-2.0) that is applied to CLA Assistant itself.
92
-
In particular you need to agree to the Individual Contributor License Agreement,
93
-
which can be [found here](https://gist.github.com/CLAassistant/bd1ea8ec8aa0357414e8).
94
-
(This applies to all contributors, including those contributing on behalf of a company). If you agree to its content, you simply have to click on the link posted by the CLA assistant as a comment to the pull request. Click it to check the CLA, then accept it on the following screen if you agree to it. CLA assistant will save this decision for upcoming contributions and will notify you if there is any change to the CLA in the meantime.
95
-
96
-
#### Company Contributors
97
-
98
-
If employees of a company contribute code, in **addition** to the individual agreement above, there needs to be one company agreement submitted. This is mainly for the protection of the contributing employees.
99
-
100
-
A company representative authorized to do so needs to download, fill, and print
101
-
the [Corporate Contributor License Agreement](/SAP Corporate Contributor License Agreement (5-26-15).pdf) form. Then either:
102
-
103
-
- Scan it and e-mail it to [opensource@sap.com](mailto:opensource@sap.com) and [tools@sap.com](mailto:tools@sap.com)
104
-
- Send it by traditional letter to: *SAP SE, Industry Standards & Open Source Team, Dietmar-Hopp-Allee 16, 69190 Walldorf, Germany*
88
+
## Developer Certificate of Origin (DCO)
105
89
90
+
Due to legal reasons, contributors will be asked to accept a DCO before they submit the first pull request to this projects, this happens in an automated fashion during the submission process. SAP uses [the standard DCO text of the Linux Foundation](https://developercertificate.org/).
106
91
107
92
### Contribution Content Guidelines
108
93
@@ -111,8 +96,8 @@ These are some of the rules we try to follow:
111
96
- Apply a clean coding style adapted to the surrounding code, even though we are aware the existing code is not fully clean
112
97
- Use (4)spaces for indentation (except if the modified file consistently uses tabs)
113
98
- Use variable naming conventions like in the other files you are seeing (camelcase)
114
-
- No console.log() - instead use GitHub Action Native logging service (For example `core.info("Hello World")`)
115
-
- Run the TSLint code check and make it succeed
99
+
- No console.log() - use logging service
100
+
- Run the ESLint code check and make it succeed
116
101
- Comment your code where it gets non-trivial
117
102
- Keep an eye on performance and memory consumption, properly destroy objects when not used anymore
0 commit comments