Skip to content

Config.resolved: Major bugfix(es)#249

Merged
bors[bot] merged 1 commit intomasterfrom
config/resolved-bug
Apr 28, 2021
Merged

Config.resolved: Major bugfix(es)#249
bors[bot] merged 1 commit intomasterfrom
config/resolved-bug

Conversation

@nicoonoclaste
Copy link
Collaborator

No description provided.

- returned prematurely on the first Path key
- raised `TypeError` if `self.ignore` did not contain a relative path
- possible confusion between keys that take one path, and ones taking sets of paths
@nicoonoclaste
Copy link
Collaborator Author

I'm not sure what went through my mind when I wrote that code, but it was broken
as hecc. It's also very concerning that our testing did not catch it at all, but
I don't want to hold this bugfix on a testsuite improvement.

@duckinator
Copy link
Owner

This looks good to me, so I'll go ahead and merge it. Can you explain more on what the problem was and/or open an issue?

bors r+

@bors
Copy link
Contributor

bors bot commented Apr 28, 2021

Build succeeded:

@bors bors bot merged commit 324be52 into master Apr 28, 2021
@bors bors bot deleted the config/resolved-bug branch April 28, 2021 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants