Skip to content

tests: cleanup user configuration.#10445

Merged
MikeMcQuaid merged 2 commits intoHomebrew:masterfrom
MikeMcQuaid:rspec_tmpdir_parallel
Jan 28, 2021
Merged

tests: cleanup user configuration.#10445
MikeMcQuaid merged 2 commits intoHomebrew:masterfrom
MikeMcQuaid:rspec_tmpdir_parallel

Conversation

@MikeMcQuaid
Copy link
Copy Markdown
Member

@MikeMcQuaid MikeMcQuaid commented Jan 28, 2021

I suspect this may be part of the cause of the flaky cleanup spec.

Also, while we're working on that, revert 3724c73 because it didn't make any different to the cleanup spec.

@BrewTestBot
Copy link
Copy Markdown
Contributor

Review period will end on 2021-01-29 at 09:04:56 UTC.

@BrewTestBot BrewTestBot added the waiting for feedback Merging is blocked until sufficient time has passed for review label Jan 28, 2021
@MikeMcQuaid MikeMcQuaid added the critical Critical change which should be shipped as soon as possible. label Jan 28, 2021
@BrewTestBot BrewTestBot removed the waiting for feedback Merging is blocked until sufficient time has passed for review label Jan 28, 2021
@BrewTestBot
Copy link
Copy Markdown
Contributor

Review period skipped due to critical label.

@MikeMcQuaid MikeMcQuaid changed the title test/support/lib/config: put test number in tmpdir path. tests: cleanup user configuration. Jan 28, 2021
@MikeMcQuaid MikeMcQuaid merged commit 5f66e15 into Homebrew:master Jan 28, 2021
@MikeMcQuaid MikeMcQuaid deleted the rspec_tmpdir_parallel branch January 28, 2021 11:38
@BrewTestBot BrewTestBot added the outdated PR was locked due to age label Feb 28, 2021
@Homebrew Homebrew locked as resolved and limited conversation to collaborators Feb 28, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

critical Critical change which should be shipped as soon as possible. outdated PR was locked due to age

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants