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

cleanup: update metadata.json license #118

Merged
merged 1 commit into from Jan 27, 2023
Merged

cleanup: update metadata.json license #118

merged 1 commit into from Jan 27, 2023

Conversation

b4ldr
Copy link
Member

@b4ldr b4ldr commented Jan 27, 2023

This PR rupdates the licence in the metadata.json file to match the LICENCE file in the repo

@dalen can you just confirm this PR, i wouldn't want to accidentally change your licencing intentions, thanks

Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

Normally we'd include the dropping of the lock file in #117, since our modules require that.

metadata.json Show resolved Hide resolved
@ekohl
Copy link
Member

ekohl commented Jan 27, 2023

I'd also suggest to disable Rubocop initially and fix it up later. We have https://github.com/voxpupuli/gha-puppet#basic-tests, but don't expose this in our modulesync config. I'd just patch it up manually now.

This PR removes updates the licence in the
metadata.json file to match the LICENCE file in the repo
@b4ldr b4ldr changed the title cleanup: drop lock file and update metadata.json licence cleanup: update metadata.json licence Jan 27, 2023
@bastelfreak bastelfreak changed the title cleanup: update metadata.json licence cleanup: update metadata.json license Jan 27, 2023
@dalen
Copy link
Contributor

dalen commented Jan 27, 2023

That's fine. The expat license is one of the variants of the MIT license. Without the advertisement clause etc.

But I think MIT license as defined by OSI is identical to expat license.

@b4ldr
Copy link
Member Author

b4ldr commented Jan 27, 2023

@dalen awesome thanks

@b4ldr b4ldr merged commit 85c1ee8 into master Jan 27, 2023
@b4ldr b4ldr deleted the clean_up branch January 27, 2023 16:30
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.

None yet

4 participants