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

chore: update NOTICE #1138

Merged
merged 3 commits into from
Sep 5, 2022
Merged

chore: update NOTICE #1138

merged 3 commits into from
Sep 5, 2022

Conversation

kenji-miyake
Copy link
Contributor

@kenji-miyake kenji-miyake commented Jun 21, 2022

Description

I updated NOTICE for AutoCore and Leo Drive since it seems they started using the format of The Autoware Contributors. (For example in #1137.)

Related: autowarefoundation/autoware#2425

The format is from https://github.com/apache/solr/blob/0ba432371064586a900ae1e66b483f0685d4ef7f/NOTICE.txt.

@Sharrrrk @xmfcx Could you check if your license notations are appropriate? I've checked some of your files, but I'm not confident.

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

NOTICE Outdated Show resolved Hide resolved
Co-authored-by: M. Fatih Cırıt <xmfcx@users.noreply.github.com>
@xmfcx
Copy link
Contributor

xmfcx commented Jun 21, 2022

@Sharrrrk could you also confirm the AutoCore part please?

@kenji-miyake
Copy link
Contributor Author

@Sharrrrk Friendly ping. 🙇

@kenji-miyake
Copy link
Contributor Author

@Sharrrrk Could you review this, please?

NOTICE Outdated Show resolved Hide resolved
Copy link
Contributor

@Sharrrrk Sharrrrk left a comment

Choose a reason for hiding this comment

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

All good.

@kenji-miyake kenji-miyake enabled auto-merge (squash) September 5, 2022 05:41
@kenji-miyake kenji-miyake merged commit c10879c into main Sep 5, 2022
@kenji-miyake kenji-miyake deleted the update-notice branch September 5, 2022 06:39
boyali referenced this pull request in boyali/autoware.universe Sep 28, 2022
* chore: update NOTICE

* Update NOTICE

Co-authored-by: M. Fatih Cırıt <xmfcx@users.noreply.github.com>

* Update NOTICE

Co-authored-by: M. Fatih Cırıt <xmfcx@users.noreply.github.com>
boyali referenced this pull request in boyali/autoware.universe Oct 3, 2022
* chore: update NOTICE

* Update NOTICE

Co-authored-by: M. Fatih Cırıt <xmfcx@users.noreply.github.com>

* Update NOTICE

Co-authored-by: M. Fatih Cırıt <xmfcx@users.noreply.github.com>
boyali referenced this pull request in boyali/autoware.universe Oct 3, 2022
* chore: update NOTICE

* Update NOTICE

Co-authored-by: M. Fatih Cırıt <xmfcx@users.noreply.github.com>

* Update NOTICE

Co-authored-by: M. Fatih Cırıt <xmfcx@users.noreply.github.com>
boyali referenced this pull request in boyali/autoware.universe Oct 19, 2022
* chore: update NOTICE

* Update NOTICE

Co-authored-by: M. Fatih Cırıt <xmfcx@users.noreply.github.com>

* Update NOTICE

Co-authored-by: M. Fatih Cırıt <xmfcx@users.noreply.github.com>
satoshi-ota pushed a commit to satoshi-ota/autoware.universe that referenced this pull request Feb 15, 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.

3 participants