Skip to content

Conversation

RasmusWL
Copy link
Member

Followup to #5248 and #5251

Mostly just because it's nice. But now we can avoid having the same `options`
files for the tests.
I have no clue why this changed, but since it's only the `..` part, I guess
we'll live with it
@RasmusWL RasmusWL requested a review from a team as a code owner February 24, 2021 09:14
@RasmusWL RasmusWL added the no-change-note-required This PR does not need a change note label Feb 24, 2021
Apparently CI is able to produce the ../ path, I have absolutely no clue what is
goign on...
Copy link
Contributor

@tausbn tausbn left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@tausbn tausbn merged commit d326d40 into github:main Feb 25, 2021
@RasmusWL RasmusWL deleted the test-cleanup branch February 25, 2021 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-change-note-required This PR does not need a change note Python
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants