Skip to content
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

Acheive REUSE compliance #35

Merged
merged 34 commits into from
Oct 24, 2023
Merged

Acheive REUSE compliance #35

merged 34 commits into from
Oct 24, 2023

Conversation

nspin
Copy link
Member

@nspin nspin commented Oct 21, 2023

Achieve REUSE compliance by adding SPDX license identifiers to source files.

This PR also adds a REUSE compliance check to CI via fsfe/reuse-action@v2.

In pursuit of compliance, some modules were re-organized or otherwise modified.

In pursuit of REUSE-compliance

Signed-off-by: Nick Spinale <[email protected]>
Signed-off-by: Nick Spinale <[email protected]>
To make way for REUSE

Signed-off-by: Nick Spinale <[email protected]>
.github/workflows/push.yaml Outdated Show resolved Hide resolved
Copy link
Member

@lsf37 lsf37 left a comment

Choose a reason for hiding this comment

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

Thanks for doing that. I haven't reviewed every single file, but everything I have looked at looks good.

Only minor request for change is to use the central seL4 action for the check so that the different repos don't diverge in the future.

@nspin nspin marked this pull request as ready for review October 24, 2023 00:38
As part of more a detailed pass.

Signed-off-by: Nick Spinale <[email protected]>
As part of more a detailed pass.

Signed-off-by: Nick Spinale <[email protected]>
As part of more a detailed pass.

Signed-off-by: Nick Spinale <[email protected]>
As part of more a detailed pass.

Signed-off-by: Nick Spinale <[email protected]>
@nspin nspin merged commit 9acd917 into seL4:main Oct 24, 2023
15 checks passed
@nspin nspin deleted the pr/reuse-compliance branch October 24, 2023 03:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants