Skip to content

Don't delete fine-grained deps cache when a module has an error - #6333

Merged
msullivan merged 1 commit into
masterfrom
fix-incremental-fg
Feb 4, 2019
Merged

msullivan merged 1 commit into
masterfrom
fix-incremental-fg

Conversation

@msullivan

Copy link
Copy Markdown
Collaborator

The dependencies are created by other modules, so it is wrong to throw
away the dependency information, since it won't be recreated when the
module is fixed and rechecked.

Previously the behavior was to crash, which is bad.

The dependencies are created by other modules, so it is wrong to throw
away the dependency information, since it won't be recreated when the
module is fixed and rechecked.

Previously the behavior was to *crash*, which is bad.
@msullivan
msullivan requested a review from gvanrossum February 4, 2019 22:43
@msullivan msullivan mentioned this pull request Feb 4, 2019
@msullivan
msullivan merged commit 34ea2af into master Feb 4, 2019
@msullivan
msullivan deleted the fix-incremental-fg branch February 4, 2019 23:25
gvanrossum pushed a commit to gvanrossum/mypy that referenced this pull request Feb 5, 2019
…on#6333)

The dependencies are created by other modules, so it is wrong to throw
away the dependency information, since it won't be recreated when the
module is fixed and rechecked.

Previously the behavior was to *crash*, which is bad.
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