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

add LICENSE amendments as suggested #59

Merged
merged 6 commits into from
Oct 1, 2019
Merged

add LICENSE amendments as suggested #59

merged 6 commits into from
Oct 1, 2019

Conversation

dcousens
Copy link
Member

@dcousens dcousens commented Sep 23, 2019

An attempt at following up #43

@dcousens
Copy link
Member Author

@calvinmetcalf I made an attempt, but it'd be good if you could verify if these attributions and LICENSE's match the derivations that you made.

I am not a lawyer, merely trying to ensure credit is given where required.

@dcousens
Copy link
Member Author

@bastien-roucaries if you could review this too that'd be great!

@dcousens dcousens marked this pull request as ready for review September 23, 2019 10:51
@calvinmetcalf
Copy link
Contributor

i did this a very long time ago but this seems fine

@dcousens
Copy link
Member Author

@calvinmetcalf if you think its' good, merge + publish! I am not part of the org 👍

@calvinmetcalf calvinmetcalf merged commit e107f57 into browserify:master Oct 1, 2019
@ljharb ljharb mentioned this pull request Sep 18, 2023
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