Skip to content

Update CONTRIBUTING.md with information about DCO #7397

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jul 17, 2019
Merged

Update CONTRIBUTING.md with information about DCO #7397

merged 3 commits into from
Jul 17, 2019

Conversation

techknowlogick
Copy link
Member

@techknowlogick techknowlogick added the type/docs This PR mainly updates/creates documentation label Jul 9, 2019
@@ -77,7 +77,7 @@ Here's how to run the test suite:
creates (a default AWS or GCE disk size won't work -- see
[#6243](https://github.com/go-gitea/gitea/issues/6243)).
- Change into the base directory of your copy of the gitea repository,
and run `drone exec --local --build-event pull_request`.
and run `drone exec --event pull_request`.
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line is unrelated to this PR, but as it is small rather than open a second PR for just this I have included here. If there is opposition to including this line in this PR please comment here and I will open a second PR for this line.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think there is an open issue related to that. I will look into it.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was closed by an other PR but this one complete it #7350

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jul 9, 2019
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Jul 9, 2019
@sapk
Copy link
Member

sapk commented Jul 9, 2019

The build failure /bin/sh: 1: ./migrations.test: Text file busy seems not related.

@zeripath
Copy link
Contributor

zeripath commented Jul 9, 2019

Otherwise lgtm

Co-Authored-By: zeripath <[email protected]>
CONTRIBUTING.md Outdated
If you set your `user.name` and `user.email` git configs, you can add the
line to the end of your commit automatically with `git commit -s`.

We assume in good faith that the information you provide is legally biding.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
We assume in good faith that the information you provide is legally biding.
We assume in good faith that the information you provide adheres to the [DCO](DCO), and won't request more personal information than provided for contributions.

Not sure if this change makes sense with the DCO, but also we do allow pseudonyms and we probably aren't assuming they are 100% legally binding (and in many cases as the contributions aren't on a large scale to wonder if they were taken from anywhere else). Also I think with the information above most people who don't mind will do the sign off with legal name as they do now and people who don't want to won't

@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Jul 17, 2019
@techknowlogick techknowlogick merged commit f1ca8ed into go-gitea:master Jul 17, 2019
@techknowlogick techknowlogick deleted the dco branch July 17, 2019 20:03
@codecov-io
Copy link

Codecov Report

Merging #7397 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #7397      +/-   ##
==========================================
- Coverage   41.23%   41.22%   -0.01%     
==========================================
  Files         469      469              
  Lines       63602    63602              
==========================================
- Hits        26224    26223       -1     
+ Misses      33959    33958       -1     
- Partials     3419     3421       +2
Impacted Files Coverage Δ
modules/process/manager.go 76.81% <0%> (-4.35%) ⬇️
modules/log/event.go 64.61% <0%> (-1.03%) ⬇️
models/repo_list.go 72.08% <0%> (-1.02%) ⬇️
routers/repo/view.go 43.25% <0%> (+1.01%) ⬆️
models/unit.go 67.56% <0%> (+5.4%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c8aa46d...ee7259d. Read the comment docs.

jeffliu27 pushed a commit to jeffliu27/gitea that referenced this pull request Jul 18, 2019
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/docs This PR mainly updates/creates documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants