MobilityData / gtfs-validator

Canonical GTFS Validator project for schedule (static) files.
https://gtfs-validator.mobilitydata.org/
Apache License 2.0
266 stars 100 forks source link

feat: Add examples table for mixed_case_recommended_field #1708

Closed emmambd closed 3 months ago

emmambd commented 3 months ago

Summary:

Closes #1625 to

Expected behavior:

Includes table of good and bad examples related to this notice.

Please make sure these boxes are checked before submitting your pull request - thanks!

Screenshot 2024-03-12 at 11 26 32 AM Screenshot 2024-03-13 at 11 25 29 AM

Testing:

Example feed

github-actions[bot] commented 3 months ago

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

davidgamez commented 3 months ago

This is how the mixed case notice looks like with a HTML caption tag on the table Rendering on the web:

Screenshot 2024-03-12 at 3 03 14 PM

Rendering on local HTML report:

Screenshot 2024-03-12 at 3 08 45 PM
emmambd commented 3 months ago

Modified HTML report style:

Screenshot 2024-03-12 at 3 55 46 PM
github-actions[bot] commented 3 months ago

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

github-actions[bot] commented 3 months ago

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

github-actions[bot] commented 3 months ago

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