PolicyEngine / policyengine-core

Core microsimulation engine for PolicyEngine models. Forked from OpenFisca-Core.
https://policyengine.github.io/policyengine-core
GNU Affero General Public License v3.0
14 stars 19 forks source link

Test that .pdf references in parameters have a page number #110

Open MaxGhenis opened 1 year ago

MaxGhenis commented 1 year ago

I'd like to enforce via CI all pdf references in parameters including a page number via the .pdf#page=x URL.

martinholmer commented 1 year ago

@MaxGhenis, Good idea. But how about enforcing the "make format" via CI. All kinds of PRs are being merged with "line" failures. It is a real hassle for other developers who when updating their branch find "make format" errors inherited from prior merged PRs.

MaxGhenis commented 1 year ago

See https://github.com/PolicyEngine/policyengine-us/issues/2064