-
Notifications
You must be signed in to change notification settings - Fork 0
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
update license to BSD 3-clause #61
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
One minor point. We have now a mixture of Crown Copyright and British Crown Copyright. Worth making them consistent?
(Our branding team will argue for Crown Copyright instead of British Crown Copyright. For international collaboration, however, British Crown Copyright is likely to be more diplomatic. The choice is yours. 😄)
@matthewrmshin Thanks! I hadn't noticed that it was all a bit of a mismatch. Hopefully I have caught all of those and converted them over now. I have removed any attempts at monarchy erasure for the Dutch, Spanish etc 😄 |
e0cdaee
to
bb893a3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Testing in my environment to make sure it builds OK. Will approve when done.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested in my environment. Works as expected.
Description
Update the license to BSD 3-clause to match that used by other public met office packages such as MetOffice/opsinputs/pull/167
Issue(s) addressed
Part of #60
Impact
None expected.
Checklist