Skip to content

Combine internal cargo features #8257

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
merged 2 commits into from
Jan 10, 2022
Merged

Conversation

camsteffen
Copy link
Contributor

changelog: none

This is just simpler and I don't see any downsides.

@rust-highfive
Copy link

r? @giraffate

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Jan 10, 2022
@xFrednet
Copy link
Member

The features were split up for the development, now they can be merged. I at least see no downside. 👍 From me

@giraffate
Copy link
Contributor

@bors r+

Thanks!

@bors
Copy link
Contributor

bors commented Jan 10, 2022

📌 Commit e2ce4f9 has been approved by giraffate

@bors
Copy link
Contributor

bors commented Jan 10, 2022

⌛ Testing commit e2ce4f9 with merge 1816361...

@bors
Copy link
Contributor

bors commented Jan 10, 2022

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: giraffate
Pushing 1816361 to master...

@bors bors merged commit 1816361 into rust-lang:master Jan 10, 2022
@camsteffen camsteffen deleted the internal-features branch January 10, 2022 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants