Skip to content

Deploy validation fixes to production #978

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

Merged
merged 24 commits into from
Mar 31, 2021
Merged

Deploy validation fixes to production #978

merged 24 commits into from
Mar 31, 2021

Conversation

krivard
Copy link
Contributor

@krivard krivard commented Mar 31, 2021

Description

The validation problems that came up over the weekend have been fixed and are ready to replace their corresponding production patches.

Other changes along for the ride.

Changelog

Itemize code/test/documentation changes and files added/removed.

  • #976, Update params documentation
  • #963, [fb-package] Add in-person schooling aggs to API
  • #971, Pull run_module from indicator.run rather than indicator in indicator runner
  • #975, [fb-package] Create function to perform multinomial Jeffreys correction
  • #967, Enable setting end date in Google Symptoms
  • #974, Remove excess printing from the validator
  • #972, Add --dry_run flag to validator

nmdefries and others added 24 commits March 25, 2021 12:43
Add --dry_run flag to validator
Remove C0330, which has been removed from [pylint](pylint-dev/pylint#3761). Including an undefined error tag causes subsequent tags to not be recognized.
Remove excess printing from the validator
Enable setting end date in Google Symptoms
[fb-package] Create function to perform multinomial Jeffreys correction
Pull `run_module` from `indicator.run` rather than `indicator` in indicator runner
[fb-package] Add in-person schooling aggs to API
@krivard krivard merged commit c678a7f into prod Mar 31, 2021
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.

4 participants