Skip to content
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

Freeze filter arguments #881

Merged
merged 2 commits into from Jul 1, 2016
Merged

Freeze filter arguments #881

merged 2 commits into from Jul 1, 2016

Conversation

@ddfreyne
Copy link
Member

@ddfreyne ddfreyne commented Jun 30, 2016

Potential fix for #866.

Mutating the filter arguments will break the outdatedness checker (this has always been the case).

  • Test
  • Improve error message
@ddfreyne ddfreyne force-pushed the freeze-filter-args branch from 07c6c8c to 89fb5fe Jul 1, 2016
@ddfreyne ddfreyne added this to the 4.2.2 milestone Jul 1, 2016
@ddfreyne ddfreyne merged commit 9f4f1b9 into release-4.2.x Jul 1, 2016
1 check passed
Loading
@ddfreyne ddfreyne deleted the freeze-filter-args branch Jul 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant