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

Deprecate periodic input ... #626

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jbusecke
Copy link
Contributor

This is my attempt to resolve a bunch of issues that stem from the fact that we still support periodic as input. This should stop right here.

This is a reimplementation of #534 which is quite far behind at this point (my apologies to the original author for falling behind on the maintainer duties here).

Totally just taking a first stab here to give people a point of reference. This needs some careful (not past 9pm) consideration of how to change the tests. The fact that periodic is so deeply embedded into the test suite is really the reason that it was not removed earlier.

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.

boundary="periodic"?
1 participant