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

[Merged by Bors] - wgpu has been moved from "wgpu-rs" to "wgpu" #2445

Closed
wants to merge 1 commit into from
Closed

Conversation

temhotaokeaha
Copy link
Contributor

Name and link update

@github-actions github-actions bot added the S-Needs-Triage This issue needs to be labelled label Jul 9, 2021
@mockersf mockersf added C-Docs An addition or correction to our documentation and removed S-Needs-Triage This issue needs to be labelled labels Jul 9, 2021
@NiklasEi
Copy link
Member

NiklasEi commented Jul 9, 2021

@mockersf can you trigger the workflow? I don't think I can.

@mockersf mockersf added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Jul 9, 2021
@mockersf
Copy link
Member

mockersf commented Jul 9, 2021

bors r+

bors bot pushed a commit that referenced this pull request Jul 9, 2021
@mockersf
Copy link
Member

mockersf commented Jul 9, 2021

@NiklasEi no one but Cart can trigger the workflow directly, but you should be able to run bors try

@bors bors bot changed the title wgpu has been moved from "wgpu-rs" to "wgpu" [Merged by Bors] - wgpu has been moved from "wgpu-rs" to "wgpu" Jul 9, 2021
@bors bors bot closed this Jul 9, 2021
@DJMcNab
Copy link
Member

DJMcNab commented Jul 16, 2021

@temhotaokeaha please respond in #2373 for the relicense to MIT/Apache 2.0. Thanks!

bors bot pushed a commit that referenced this pull request Jul 17, 2021
# Objective

Currently we can sometimes allow PRs by people who haven't agreed to the relicense to get merged into main.

E.g. #2445

## Solution

This adds a check to ensure that this doesn't happen, by ensuring that bors doesn't relicense said PRs.

As a bonus, it also adds config to automatically label new PRs as needing checking, to ensure that we do the verification until we merge the new license.
DJMcNab added a commit to DJMcNab/bevy that referenced this pull request Jul 17, 2021
This makes more sense when we are talking
about the wgpu project as a whole

Obsoletes bevyengine#2445
bors bot pushed a commit that referenced this pull request Jul 17, 2021
This obsoletes #1111 and #2445, since @ColonisationCaptain and @temhotaokeaha haven't replied to #2373.

I believe that both of those PRs would be fine to keep, but they're even more fine to keep now :)
ostwilkens pushed a commit to ostwilkens/bevy that referenced this pull request Jul 27, 2021
ostwilkens pushed a commit to ostwilkens/bevy that referenced this pull request Jul 27, 2021
# Objective

Currently we can sometimes allow PRs by people who haven't agreed to the relicense to get merged into main.

E.g. bevyengine#2445

## Solution

This adds a check to ensure that this doesn't happen, by ensuring that bors doesn't relicense said PRs.

As a bonus, it also adds config to automatically label new PRs as needing checking, to ensure that we do the verification until we merge the new license.
ostwilkens pushed a commit to ostwilkens/bevy that referenced this pull request Jul 27, 2021
This obsoletes bevyengine#1111 and bevyengine#2445, since @ColonisationCaptain and @temhotaokeaha haven't replied to bevyengine#2373.

I believe that both of those PRs would be fine to keep, but they're even more fine to keep now :)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Docs An addition or correction to our documentation S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants