Skip to content

UI Test Cleanup: Split out out_of_bounds_indexing #4278

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Jul 17, 2019
Merged

UI Test Cleanup: Split out out_of_bounds_indexing #4278

merged 5 commits into from
Jul 17, 2019

Conversation

phansch
Copy link
Member

@phansch phansch commented Jul 16, 2019 •
edited
Loading

This moves the out_of_bounds_indexing lint tests to their own
directory.

changelog: none

cc #2038

@phansch
UI Test Cleanup: Split out out_of_bounds_indexing
33010ae 
This moves the `out_of_bounds_indexing` lint tests to their own
directory.
flip1995
flip1995 reviewed Jul 16, 2019
View reviewed changes
flip1995
flip1995 reviewed Jul 16, 2019
View reviewed changes
flip1995
flip1995 reviewed Jul 16, 2019
View reviewed changes
@flip1995
Copy link
Member

./util/dev fmt

@matthiaskrgr matthiaskrgr added the S-waiting-on-author Status: This is awaiting some action from the author. (Use `@rustbot ready` to update this status) label Jul 16, 2019
phansch and others added 3 commits July 16, 2019 21:28
@phansch
cargo fmt
633934d
@phansch @flip1995
Remove comment
65e9477 
Co-Authored-By: Philipp Krones <[email protected]>
@phansch @flip1995
Allow no_effect lint
7a88896 
for cleaner stderr file

Co-Authored-By: Philipp Krones <[email protected]>
@phansch phansch added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties and removed S-waiting-on-author Status: This is awaiting some action from the author. (Use `@rustbot ready` to update this status) labels Jul 16, 2019
@flip1995
Copy link
Member

Thanks!

@bors r+

@bors
Copy link
Contributor

bors commented Jul 17, 2019

📌 Commit 38b6156 has been approved by flip1995

@bors
Copy link
Contributor

bors commented Jul 17, 2019

⌛ Testing commit 38b6156 with merge fb35311...

bors added a commit that referenced this pull request Jul 17, 2019
@bors
Auto merge of #4278 - phansch:uitestcleanup_indexing, r=flip1995
fb35311 
UI Test Cleanup: Split out out_of_bounds_indexing

This moves the `out_of_bounds_indexing` lint tests to their own
directory.

changelog: none

cc #2038
@bors
Copy link
Contributor

bors commented Jul 17, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: flip1995
Pushing fb35311 to master...

@bors bors merged commit 38b6156 into rust-lang:master Jul 17, 2019
@phansch phansch deleted the uitestcleanup_indexing branch July 17, 2019 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@flip1995 flip1995 flip1995 approved these changes

Assignees
No one assigned
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@phansch @flip1995 @bors @matthiaskrgr