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

Use SPDX identifier in license field of META6.json #78

Merged
merged 1 commit into from Apr 26, 2017

Conversation

samcv
Copy link
Contributor

@samcv samcv commented Apr 26, 2017

Use the standardized identifier for the license field.
For more details see https://design.perl6.org/S22.html#license

@teodozjan
Copy link
Contributor

I think it would be nice to add newline at EOF

@toolforger
Copy link

Another nitpick (or maybe not, don't know what the Perl6 take on these things is):
It would be nice to put the reformatting into one commit, the rearrangement into another, and the actual change into a third.
I was unable to see what was actually done, if I were to review this I'd be in trouble.

@samcv
Copy link
Contributor Author

samcv commented Apr 26, 2017

Ran my 2.0 version script on it. The 1.0 just read json and then wrote it back, the new one inserts in and spaces the license tag as well.

Use the standardized identifier for the license field.
For more details see https://design.perl6.org/S22.html#license
@teodozjan teodozjan merged commit f8ac1b9 into finanalyst:master Apr 26, 2017
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

3 participants