Skip to content

Clarify align_to's requirements and obligations - #63293

Merged
bors merged 1 commit into
rust-lang:masterfrom
shepmaster:align-to-doc
Aug 6, 2019
Merged

Clarify align_to's requirements and obligations#63293
bors merged 1 commit into
rust-lang:masterfrom
shepmaster:align-to-doc

Conversation

@shepmaster

Copy link
Copy Markdown
Member

Based on discussions with @RalfJung around my misunderstandings when using this.

r? @RalfJung

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 5, 2019
@RalfJung

Copy link
Copy Markdown
Member

Thanks!

@bors r+ rollup

@bors

bors commented Aug 5, 2019

Copy link
Copy Markdown
Collaborator

📌 Commit 7ad0e0e3446c1ed4a80f8c8d772b5d4808471afd has been approved by RalfJung

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 5, 2019
Comment threadsrc/libcore/slice/mod.rs Outdated
Comment threadsrc/libcore/slice/mod.rs Outdated
@Centril

Copy link
Copy Markdown
Contributor

@bors r-

Some typos :)

@borsbors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Aug 5, 2019
@shepmaster

Copy link
Copy Markdown
MemberAuthor

Thanks!

@bors r=@RalfJung

@bors

bors commented Aug 5, 2019

Copy link
Copy Markdown
Collaborator

📌 Commit 571e22d has been approved by @RalfJung

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Aug 5, 2019
Centril added a commit to Centril/rust that referenced this pull request Aug 6, 2019
Clarify align_to's requirements and obligations
Based on discussions with @RalfJung around my misunderstandings when using this.
r? @RalfJung
@CentrilCentril mentioned this pull request Aug 6, 2019
Centril added a commit to Centril/rust that referenced this pull request Aug 6, 2019
Clarify align_to's requirements and obligations
Based on discussions with @RalfJung around my misunderstandings when using this.
r? @RalfJung
@CentrilCentril mentioned this pull request Aug 6, 2019
@RalfJung

Copy link
Copy Markdown
Member

No @ for bors (yeah don't ask)

@bors r+

@bors

bors commented Aug 6, 2019

Copy link
Copy Markdown
Collaborator

💡 This pull request was already approved, no need to approve it again.

@bors

bors commented Aug 6, 2019

Copy link
Copy Markdown
Collaborator

📌 Commit 571e22d has been approved by RalfJung

Centril added a commit to Centril/rust that referenced this pull request Aug 6, 2019
Clarify align_to's requirements and obligations
Based on discussions with @RalfJung around my misunderstandings when using this.
r? @RalfJung
@CentrilCentril mentioned this pull request Aug 6, 2019
bors added a commit that referenced this pull request Aug 6, 2019
Rollup of 14 pull requests
Successful merges:
- #61457 (Implement DoubleEndedIterator for iter::{StepBy, Peekable, Take})
- #63017 (Remove special code-path for handing unknown tokens)
- #63184 (Explaining the reason why validation is performed in to_str of path.rs)
- #63230 (Make use of possibly uninitialized data [E0381] a hard error)
- #63260 (fix UB in a test)
- #63264 (Revert "Rollup merge of #62696 - chocol4te:fix_#62194, r=estebank")
- #63272 (Some more libsyntax::attr cleanup)
- #63285 (Remove leftover AwaitOrigin)
- #63287 (Don't store &Span)
- #63293 (Clarify align_to's requirements and obligations)
- #63295 (improve align_offset docs)
- #63299 (Make qualify consts in_projection use PlaceRef)
- #63312 (doc: fix broken sentence)
- #63315 (Fix#63313)
Failed merges:
r? @ghost
@bors
bors merged commit 571e22d into rust-lang:masterAug 6, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: 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.

5 participants

@shepmaster@RalfJung@bors@Centril@rust-highfive