Skip to content

Tolerate hidden, binary files in tests/ #11905

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

Merged
merged 1 commit into from
Dec 2, 2023
Merged

Conversation

pgerber
Copy link
Contributor

@pgerber pgerber commented Dec 2, 2023

Avoid scanning temporary files created by editors like this one created by Vim:

---- old_test_headers stdout ----
thread 'old_test_headers' panicked at tests/headers.rs:19:74: tests/ui/.regex.rs.swp: stream did not contain valid UTF-8 note: run with RUST_BACKTRACE=1 environment variable to display a backtrace

changelog: none

Avoid scanning temporary files created by editors like
this one created by Vim:

---- old_test_headers stdout ----
thread 'old_test_headers' panicked at tests/headers.rs:19:74:
tests/ui/.regex.rs.swp: stream did not contain valid UTF-8
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
@rustbot
Copy link
Collaborator

rustbot commented Dec 2, 2023

r? @dswij

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

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Dec 2, 2023
Copy link
Member

@dswij dswij left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@dswij
Copy link
Member

dswij commented Dec 2, 2023

@bors r+

@bors
Copy link
Contributor

bors commented Dec 2, 2023

📌 Commit 1e67f6c has been approved by dswij

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Dec 2, 2023

⌛ Testing commit 1e67f6c with merge da27c97...

@bors
Copy link
Contributor

bors commented Dec 2, 2023

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: dswij
Pushing da27c97 to master...

@bors bors merged commit da27c97 into rust-lang:master Dec 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants