Skip to content

Add EUPL v1.2 as license #3475

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Feb 9, 2018
Merged

Add EUPL v1.2 as license #3475

merged 1 commit into from
Feb 9, 2018

Conversation

jonasfranz
Copy link
Member

Fixes #3439

@jonasfranz
Add EUPL v1.2 as license (Fixes go-gitea#3439)
f7f02a2 
Signed-off-by: Jonas Franz <info@jonasfranz.de>
@codecov-io
Copy link

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3475      +/-   ##
==========================================
- Coverage   35.68%   35.68%   -0.01%     
==========================================
  Files         281      281              
  Lines       40708    40708              
==========================================
- Hits        14527    14525       -2     
- Misses      24030    24032       +2     
  Partials     2151     2151
Impacted Files Coverage Δ
models/repo_list.go 65.62% <0%> (-1.57%) ⬇️

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 011f128...f7f02a2. Read the comment docs.

@tboerger tboerger added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Feb 8, 2018
@lunny lunny added this to the 1.5.0 milestone Feb 9, 2018
@lunny lunny added the type/docs This PR mainly updates/creates documentation label Feb 9, 2018
@lunny
Copy link
Member

lunny commented Feb 9, 2018

LGTM

@tboerger tboerger 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 Feb 9, 2018
@tboerger tboerger 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 Feb 9, 2018
@lafriks lafriks merged commit 58771ac into go-gitea:master Feb 9, 2018
@bkcsoft
Copy link
Member

bkcsoft commented Feb 10, 2018

Shouldn't this be sent upstream to https://github.com/spdx/license-list instead? Now we're breaking the use of scripts/generate-licenses.go 😒

@jonasfranz
Copy link
Member Author

@bkcsoft generate-licenses is already broken because https://github.com/spdx/license-list is no longer maintained. Because https://github.com/spdx/license-list-XML is the new actual list.

@jonasfranz jonasfranz mentioned this pull request Feb 10, 2018
Merged
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
1 more reviewer

@sapk sapk sapk approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
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
Milestone
1.5.0
Development

Successfully merging this pull request may close these issues.

Add EUPL v1.2 as license
7 participants
@jonasfranz @codecov-io @lunny @bkcsoft @sapk @tboerger @lafriks