Skip to content

Minor documentation changes#1658

Merged
jcpitre merged 3 commits intomasterfrom
1292-minor-documentation-changes-for-the-date-feature
Jan 25, 2024
Merged

Minor documentation changes#1658
jcpitre merged 3 commits intomasterfrom
1292-minor-documentation-changes-for-the-date-feature

Conversation

@jcpitre
Copy link
Copy Markdown
Contributor

@jcpitre jcpitre commented Jan 24, 2024

Summary:

Added a little bit of documentation related to #1628

@jcpitre jcpitre linked an issue Jan 24, 2024 that may be closed by this pull request
Allow the validator to accept a "time of validation" argument #1292
Closed
emmambd
emmambd reviewed Jan 24, 2024
View reviewed changes
docs/USAGE.md Outdated
| `-n` | `--export_notices_schema` | Optional | Export notice schema as a json file. |
| `-p` | `--pretty` | Optional | Pretty JSON validation report. If specified, the JSON validation report will be printed using JSON Pretty print. This does not impact data parsing. |
| `-p` | `--pretty` | Optional | Pretty JSON validation report. If specified, the JSON validation report will be printed using JSON Pretty print. This does not impact data parsing. |
| `-d` | `--date` | Optional | Date to simulate when validating, in ISO_LOCAL_DATE format like '2001-01-30'. By default, the current date is used. |
Copy link
Copy Markdown
Contributor

@emmambd emmambd Jan 24, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jcpitre "Date to simulate when validating" is a bit vague. What about "The date used to validate the feed for time-based rules, e.g feed_expiration_30_days."

Copy link
Copy Markdown
Contributor Author

@jcpitre jcpitre Jan 24, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure. I was just copying the text from @bradyhunsaker's changes, but less vague is always good.

davidgamez
davidgamez approved these changes Jan 24, 2024
View reviewed changes
Copy link
Copy Markdown
Member

@davidgamez davidgamez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 💪 !

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jan 24, 2024

✅ Rule acceptance tests passed.
New Errors: 0 out of 1481 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
Dropped Errors: 0 out of 1481 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
New Warnings: 0 out of 1481 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
Dropped Warnings: 0 out of 1481 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
0 out of 1481 sources (~0 %) are corrupted.
Commit: 56aac14
Download the full acceptance test report here (report will disappear after 90 days).
✅ Rule acceptance tests passed.

@bradyhunsaker
Copy link
Copy Markdown
Contributor

bradyhunsaker commented Jan 25, 2024

Sorry that I overlooked updating this file with the original change.

@jcpitre
Copy link
Copy Markdown
Contributor Author

jcpitre commented Jan 25, 2024 •
edited
Loading

Sorry that I overlooked updating this file with the original change.

@bradyhunsaker You did a lot already. I'm happy to take care of small details like this

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jan 25, 2024

✅ Rule acceptance tests passed.
New Errors: 0 out of 1481 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
Dropped Errors: 0 out of 1481 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
New Warnings: 0 out of 1481 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
Dropped Warnings: 0 out of 1481 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
0 out of 1481 sources (~0 %) are corrupted.
Commit: f27ba14
Download the full acceptance test report here (report will disappear after 90 days).
✅ Rule acceptance tests passed.

@jcpitre jcpitre merged commit 48facc3 into master Jan 25, 2024
@jcpitre jcpitre deleted the 1292-minor-documentation-changes-for-the-date-feature branch January 25, 2024 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@emmambd emmambd emmambd left review comments

@davidgamez davidgamez davidgamez approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Allow the validator to accept a "time of validation" argument

4 participants

@jcpitre @bradyhunsaker @davidgamez @emmambd