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 BUSL license text & bump node version #414

Closed
wants to merge 5 commits into from
Closed

Conversation

QGarchery
Copy link
Contributor

No description provided.

@QGarchery QGarchery requested a review from a team April 5, 2024 09:17
@QGarchery QGarchery self-assigned this Apr 5, 2024
@QGarchery QGarchery requested review from adhusson, Rubilmax, MerlinEgalite, MathisGD and Jean-Grimal and removed request for a team April 5, 2024 09:17
@QGarchery QGarchery marked this pull request as draft April 8, 2024 11:13
@QGarchery
Copy link
Contributor Author

Putting as draft because the dual licensing seems to be on purpose. Maybe we should append the full BUSL license as well

@QGarchery QGarchery changed the title Remove mention of BUSL in the license Add BUSL license text & bump node version Apr 9, 2024
@QGarchery QGarchery marked this pull request as ready for review April 9, 2024 10:00
@QGarchery QGarchery requested a review from MerlinEgalite April 9, 2024 10:00
Comment on lines +298 to +302
The Licensed Work is (c) 2023 Morpho Association

Additional Use Grant: Any uses listed and defined at
morpho-blue-core-license-grants.morpho.eth
Copy link
Contributor

Choose a reason for hiding this comment

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

this i guess must be updated

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah yes. Not sure how to do it, especially the "change date" which should be void I think. I guess it should be explained that the users can choose the license as well

Rubilmax
Rubilmax previously approved these changes Apr 19, 2024
@Rubilmax Rubilmax dismissed their stale review April 19, 2024 12:41

invalid

@MathisGD
Copy link
Contributor

Pending discussion on the dual licence

@QGarchery
Copy link
Contributor Author

Closing this, let's reopen when needed

@QGarchery QGarchery closed this Aug 14, 2024
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.

4 participants