Skip to content

Conversation

@DJMcNab
Copy link
Member

@DJMcNab DJMcNab commented 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 :)

DJMcNab added 2 commits July 17, 2021 22:22
This makes more sense when we are talking
about the wgpu project as a whole

Obsoletes bevyengine#2445
@github-actions github-actions bot added S-Needs-Triage This issue needs to be labelled S-Pre-Relicense This PR was made before Bevy added the Apache license. Cannot be merged or used for other work labels Jul 17, 2021
@DJMcNab DJMcNab removed the S-Pre-Relicense This PR was made before Bevy added the Apache license. Cannot be merged or used for other work label Jul 17, 2021
@cart
Copy link
Member

cart commented Jul 17, 2021

I think you mentioned the wrong person. @CleanCut has replied (and isn't the author of #1111).

@DJMcNab
Copy link
Member Author

DJMcNab commented Jul 17, 2021

Indeed, you're the second person to point that out. I've already edited it.

@cart
Copy link
Member

cart commented Jul 17, 2021

Cool cool. I think we should delete cases like this from the list instead of checking them off (to avoid thinking they agreed to the relicense).

@cart
Copy link
Member

cart commented Jul 17, 2021

bors r+

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 :)
@cart
Copy link
Member

cart commented Jul 17, 2021

Eh we should add a "changes obsoleted" category and move them there.

@bors bors bot changed the title Useful changes with relicensing benefits [Merged by Bors] - Useful changes with relicensing benefits Jul 17, 2021
@bors bors bot closed this Jul 17, 2021
@cart
Copy link
Member

cart commented Jul 17, 2021

Updated #2373

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

S-Needs-Triage This issue needs to be labelled

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants