Skip to content
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

Remove $DIR replacement #12306

Merged
merged 2 commits into from
Feb 19, 2024
Merged

Remove $DIR replacement #12306

merged 2 commits into from
Feb 19, 2024

Conversation

Alexendoo
Copy link
Member

This won't cause problems because the old $DIR replacement was based on the parent of the test path, which for us is relative:

..Config::rustc(Path::new("tests").join(test_dir))

The new pattern being "tests/{test_dir}" is more clearly relative

That's why we have custom filters applied to the toml/cargo tests where absolute paths do appear in the output

config
.comment_defaults
.base()
.normalize_stderr
.push((Match::from(env::current_dir().unwrap().as_path()), b"$DIR".into()));

Removing it allows clicking the paths in the terminal

changelog: none

r? @flip1995

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Feb 17, 2024
@flip1995
Copy link
Member

@bors r+ p=10

Thanks!

@bors
Copy link
Collaborator

bors commented Feb 19, 2024

📌 Commit 740d89e has been approved by flip1995

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Feb 19, 2024

⌛ Testing commit 740d89e with merge 74f611f...

@bors
Copy link
Collaborator

bors commented Feb 19, 2024

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: flip1995
Pushing 74f611f to master...

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