-
Notifications
You must be signed in to change notification settings - Fork 127
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
Reuse #130
Open
NSfsfe
wants to merge
22
commits into
AtomGraph:master
Choose a base branch
from
NSfsfe:REUSE
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Reuse #130
Conversation
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hello,
By way of introduction, I am Niharika Singhal from the Free Software Foundation Europe, and the REUSE Booster program. Thank you for contacting us and we are happy to help you out with the implementation of REUSE. After a check on your repository, I would like to propose some updates regarding your copyright and licensing information. The REUSE specifications intend to make licensing easier with best practices to display legal information through comment headers on source files that can be easily human and machine readable. You can check this screencast:
https://download.fsfe.org/videos/reuse/screencasts/reuse-tool.gif
Please keep in mind that this merge request is intended to be a practical example of how REUSE looks like.
I would also like to make some remarks to be considered to make your full project REUSE compliant.
Feel completely free to adopt these specifications by merging this pull request.
REUSE Features:
SPDX copyright and license comment headers for all relevant files (please be aware that in the current state of the repository, I was able to add headers to all the directories. However, some files might miss this feature depending on the type of the file. For example, .owl, .rdf, .rq, .jar, .trig, .env files etc..) If you are interested in implementing REUSE specification, I hope this merge request can help you to do so.
LICENSES directory in the root of the project with the licenses used on the repository:
I included in this directory, the file that contained the license for your project (Apache 2.0 only) text. If some pieces of your project use a different license, please add their license text to the LICENSES directory.
Files missing copyright and licensing information:
I noticed that your project does not have the copyright and license information displayed in a comment header. According to the REUSE specifications, the comment headers MUST contain the SPDX Tag. For this, we recommend to update the headers, or at least the SPDX-License-Identifier as in this practical example.
Please also double check if the personal information in the headers is correct and consistent, in the case of several copyright holders please update that information in the headers.
In case you want to license certain files under a different license, special attention should also be paid to that aspect and such files should contain the appropriate SPDX tag.
Please note that I used the REUSE helper tool to add the headers, so the tool automatically deleted the existing header (if any) and changed it to the SPDX format.
Image files:
I added the Apache 2.0 license to all of the image files. However, you can license other image files under different licenses as well and include the license in the LICENSES directory.
We can also do a follow up of this PR and further REUSE specification with a call. Feel free to reach me out.
Hope that helps and thank you very much for the amazing job!
Niharika Singhal - Trainee
Free Software Foundation Europe
Schönhauser Allee 6/7, 10119 Berlin, Germany | t +49-30-27595290
Registered at Amtsgericht Hamburg, VR 17030 | https://fsfe.org/support