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

Rollup of 10 pull requests #89838

Closed
wants to merge 23 commits into from
Closed

Conversation

JohnTitor
Copy link
Member

Successful merges:

Failed merges:

r? @ghost
@rustbot modify labels: rollup

Create a similar rollup

pierwill and others added 23 commits October 11, 2021 12:56
This sentence was previously incomplete.
`**node_modules` in a .gitignore is the same than
`*node_modules` or `*****node_modules`.

It matches every file whose name ends with `node_modules`,
including `not_node_modules`.

The intent here was obviously to have `**/node_modules`
which is the same than just `node_modules`.
add some more testcases

resolves rust-lang#52893
resolves rust-lang#68295
resolves rust-lang#87750
resolves rust-lang#88071

All these issues have been fixed according to glacier. Just adding a test so it can be closed.

Can anybody tell me why the github keywords do not work? 🤔
Please edit this post if you can fix it.
Edit explanation of test for nested type ascriptions

Fixes typo ("an ascribing") and removes extra.

Closes rust-lang#88233.
Add missing words in `Infallible` docs

This sentence was previously incomplete.
Improve CJK font in rustdoc

This PR includes:
- Fix unicode range of korean letters in `rustdoc.css`.
- Add WOFF2 format version of Noto Sans KR font.
- Shorten the font file name.
…ions, r=joshtriplett

Add #[must_use] to to_value conversions

`NonNull<T>::cast` snuck in when I wasn't looking. What a scamp!

Parent issue: rust-lang#89692

r? ``@joshtriplett``
…s-typo, r=joshtriplett

Fix uppercase/lowercase error

Fix rust-lang#89694 (comment)

r? ``@joshtriplett``
Fix invalid rules in .gitignore

`**node_modules` in a .gitignore is the same than
`*node_modules` or `*****node_modules`.

It matches every file whose name ends with `node_modules`,
including `not_node_modules`.

The intent here was obviously to have `**/node_modules`
which is the same than just `node_modules`.

Reference on git ignoring rules format: https://git-scm.com/docs/gitignore
Use Option::map_or instead of open coding it

``@rustbot`` modify labels +C-cleanup +T-compiler
…take-two, r=Mark-Simulacrum

Fix config.toml overflow-checks options

This a follow-up PR to rust-lang#87784.

Improvements:
* Add missing entries for overflow-checks to config.toml.example.
* Add --enable-overflow-checks-std option to configure script.
* Make rust.overflow-checks-stdoption default to rust.overflow-checks.

Also adds the missing  `--enable-debug-assertions-std `option to configure script.

r? `@Mark-Simulacrum`
cc `@jyn514`
@rustbot rustbot added the rollup A PR which is a rollup label Oct 13, 2021
@JohnTitor
Copy link
Member Author

@bors r+ p=10 rollup=never

@bors
Copy link
Contributor

bors commented Oct 13, 2021

📌 Commit 5b2deeb has been approved by JohnTitor

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Oct 13, 2021
@bors
Copy link
Contributor

bors commented Oct 13, 2021

⌛ Testing commit 5b2deeb with merge b11482027b8a2501eba27885dbdf37b209fe100b...

@bors
Copy link
Contributor

bors commented Oct 13, 2021

☀️ Test successful - checks-actions
Approved by: JohnTitor
Pushing b11482027b8a2501eba27885dbdf37b209fe100b to master...

@hellow554
Copy link
Contributor

Two hours for a "simple push to master" doesn't seem normal, does it?

@JohnTitor
Copy link
Member Author

Yup, it failed to push, re-creating a new one...

@JohnTitor JohnTitor closed this Oct 13, 2021
@JohnTitor JohnTitor deleted the rollup-xcmfy28 branch October 13, 2021 12:56
@rust-timer
Copy link
Collaborator

Finished benchmarking commit (b11482027b8a2501eba27885dbdf37b209fe100b): comparison url.

Summary: This benchmark run did not return any relevant changes.

If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf.

@rustbot label: -perf-regression

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.