feat(api): provide task guards for lazy register#282
Open
ffluk3 wants to merge 1 commit intodropbox:mainfrom
Open
feat(api): provide task guards for lazy register#282ffluk3 wants to merge 1 commit intodropbox:mainfrom
ffluk3 wants to merge 1 commit intodropbox:mainfrom
Conversation
|
|
Member
|
Hey there, looks like you haven't yet signed the CLA. Also, could you remove any formatting changes added to this PR as it just appears to be noise vs the actual feature you wanted implemented |
This update provides a way to use the basic AffectedModuleDetector task guard when leveraging the `register` api to create tasks, without needing to explicitly get the task immediately. This allows for the AMD calculation to be evaluated lazily, and avoids performing work on tasks that will not be executed.
ffluk3
force-pushed
the
task-configure-lazy
branch
from
0d7b7dc to
1af62a6
Compare
Author
|
@joshafeinberg I addressed the formatting. Working with my team to ensure we can sign the CLA but will get back to you on that! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This update provides a way to use the basic AffectedModuleDetector task guard when leveraging the
registerapi to create tasks, without needing to explicitly get the task immediately. This allows for the AMD calculation to be evaluated lazily, and avoids performing work on tasks that will not be executed.