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

Support additional messages & Dual licensing with MIT #50

Closed
wants to merge 7 commits into from

Conversation

elpiel
Copy link
Member

@elpiel elpiel commented Sep 9, 2022

  • BOD (untested)
  • GBS (untested)

@elpiel elpiel mentioned this pull request Sep 10, 2022
@Dushistov
Copy link
Collaborator

I suppose that license change PR should not include any code changes.
Because of it need at least approve from all authors that contribute code to this crate,
or at least who contribute significant amount of code.

@elpiel
Copy link
Member Author

elpiel commented Sep 10, 2022

On that note, @flxo do you mind moving your repository to the @AeroRust org as it keeps defaulting to the original repo when opening PRs?

Do you agree to change the license to dual with MIT? I have added the copyright to both files.

@Dushistov do you agree to the change?

Ps: if everybody agrees I don't see why we should separated in a different PR. It is just more convenient to have it in the PR to get the confirmation right now.

@elpiel
Copy link
Member Author

elpiel commented Sep 10, 2022

@Dushistov I will remove the other commits once I'm back on the laptop

@elpiel
Copy link
Member Author

elpiel commented Sep 10, 2022

I opened #52 for the dual license change.

@elpiel elpiel closed this Sep 10, 2022
@flxo
Copy link

flxo commented Sep 11, 2022

On that note, @flxo do you mind moving your repository to the @AeroRust org as it keeps defaulting to the original repo when opening PRs?

Do you agree to change the license to dual with MIT? I have added the copyright to both files.

@elpiel I'm perfectly fine with that. Thanks.

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.

3 participants