New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Declare affected props in outdatedness rule #1137

Merged
merged 1 commit into from Mar 27, 2017

Conversation

Projects
None yet
1 participant
@ddfreyne
Member

ddfreyne commented Mar 27, 2017

This refactoring makes an outdatedness rule not have a reason associated with it. The reason was only needed in order to determine the affected props (one or more of :raw_content, :attributes, :compiled_content, :path).

This will simplify upcoming changes to the AttributesModified reason (which’ll be adjusted to also include a set of affected attributes).

@ddfreyne ddfreyne merged commit 8d5b407 into master Mar 27, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@ddfreyne ddfreyne deleted the declare-affected-props-in-outd-rule branch Mar 27, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment