Skip to content

Rollup of 7 pull requests #86032

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 14 commits into from
Jun 5, 2021
Merged

Conversation

GuillaumeGomez
Copy link
Member

Successful merges:

Failed merges:

r? @ghost
@rustbot modify labels: rollup

Create a similar rollup

liigo and others added 14 commits May 20, 2021 14:54
In particular the `is_dir`, `is_file` and `exists` functions says that querying a file requires querying the directory. On Windows this is not normally true.
Add a map method to std::ops::range::Bound, patterned off of the method
of the same name on Option
Add a map method to Bound

Add a map method to std::ops::range::Bound, patterned off of the method
of the same name on Option.

Have left off creating a tracking issue initially, but as soon as I get the go-ahead from a reviewer I'll make that right away 😄
…fic, r=m-ou-se

Possible errors when accessing file metadata are platform specific

In particular the `is_dir`, `is_file` and `exists` functions suggests that querying a file requires querying the directory. On Windows this is not normally true.

r? `@m-ou-se`
td align attribute

This is a follow-up of rust-lang#85972. I have put this one on its own because it changes the display:

![Screenshot from 2021-06-03 21-49-11](https://user-images.githubusercontent.com/3050060/120703622-d533d280-c4b5-11eb-9519-ea1131a40bee.png)

Without align:

![Screenshot from 2021-06-03 21-49-15](https://user-images.githubusercontent.com/3050060/120703623-d5cc6900-c4b5-11eb-95f9-878d3915c7fb.png)

I also opened an issue about it: pulldown-cmark/pulldown-cmark#533. However, I'm not sure if this is the right course of action... Should we instead ignore the warning?

r? ``@jsha``
msp430 linker does not accept -znoexecstack. Set linker_is_gnu to fal…

…se as workaround for now.

Tested locally and works. Closes rust-lang#85948.
@rustbot rustbot added the rollup A PR which is a rollup label Jun 5, 2021
@GuillaumeGomez
Copy link
Member Author

@bors: r+ p=7 rollup=never

@bors
Copy link
Collaborator

bors commented Jun 5, 2021

📌 Commit 16504f6 has been approved by GuillaumeGomez

@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 Jun 5, 2021
@klensy
Copy link
Contributor

klensy commented Jun 5, 2021

Some pr's common with #86029

@bors
Copy link
Collaborator

bors commented Jun 5, 2021

⌛ Testing commit 16504f6 with merge ac3e680...

@GuillaumeGomez
Copy link
Member Author

Some pr's common with #86029

Weird, how come it wasn't listed...

@bors
Copy link
Collaborator

bors commented Jun 5, 2021

☀️ Test successful - checks-actions
Approved by: GuillaumeGomez
Pushing ac3e680 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Jun 5, 2021
@bors bors merged commit ac3e680 into rust-lang:master Jun 5, 2021
@rustbot rustbot added this to the 1.54.0 milestone Jun 5, 2021
@GuillaumeGomez GuillaumeGomez deleted the rollup-y3ij27b branch June 5, 2021 21:11
@bors bors mentioned this pull request Jun 5, 2021
4 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
merged-by-bors This PR was explicitly merged by bors. 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.

10 participants