Skip to content

librustc_errors: Remove unused annotation style OldSchoolNoteText - #61073

Merged
bors merged 1 commit into
rust-lang:masterfrom
phansch:remove_unused_annotation_style
May 23, 2019
Merged

librustc_errors: Remove unused annotation style OldSchoolNoteText#61073
bors merged 1 commit into
rust-lang:masterfrom
phansch:remove_unused_annotation_style

Conversation

@phansch

@phanschphansch commented May 23, 2019

Copy link
Copy Markdown
Contributor

I could not find any references to it and the snippet module does not
seem to be exported publicly, so I think it can be safely removed.

This was originally removed in 17bd76a and I'm not sure why it is still there.

I could not find any references to it and the `snippet` module does not
seem to be exported publicly, so I think it can be safely removed.
@rust-highfive

Copy link
Copy Markdown
Contributor

r? @matthewjasper

(rust_highfive has picked a reviewer for you, use r? to override)

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

Copy link
Copy Markdown
Contributor

@bors r+ rollup=always

@bors

bors commented May 23, 2019

Copy link
Copy Markdown
Collaborator

📌 Commit 5ed4237 has been approved by matthewjasper

@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 May 23, 2019
Centril added a commit to Centril/rust that referenced this pull request May 23, 2019
…yle, r=matthewjasper
librustc_errors: Remove unused annotation style `OldSchoolNoteText`
I could not find any references to it and the `snippet` module does not
seem to be exported publicly, so I think it can be safely removed.
This was originally removed in 17bd76a and I'm not sure why it is still there.
@CentrilCentril mentioned this pull request May 23, 2019
bors added a commit that referenced this pull request May 23, 2019
Rollup of 7 pull requests
Successful merges:
- #60981 (Bump compiler-builtins to 0.1.15)
- #61014 (Make -Zemit-artifact-notifications also emit the artifact type)
- #61043 (Disable LLVM/debug assertions in gnu-full-bootstrap)
- #61046 (Fix ICE with inconsistent macro matchers)
- #61055 (Solaris CI: Build with dilos2 stable)
- #61057 (Revert "Add implementations of last in terms of next_back on a bunch of DoubleEndedIterators.")
- #61073 (librustc_errors: Remove unused annotation style `OldSchoolNoteText`)
Failed merges:
r? @ghost
@bors
bors merged commit 5ed4237 into rust-lang:masterMay 23, 2019
@phansch
phansch deleted the remove_unused_annotation_style branch May 23, 2019 10:23
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.

4 participants

@phansch@rust-highfive@matthewjasper@bors