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.

Sign up for GitHub

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

Jump to bottom

Update linting rules as discussed #14

Merged

rkettelerij merged 2 commits into master from linting

Nov 22, 2023

Collaborator

rkettelerij commented Nov 22, 2023

Omschrijving

Een omschrijving van wat je hebt toegevoegd/veranderd:

https://dev.kadaster.nl/jira/browse/PDOK-

Type verandering

(Verwijder de opties die niet relevant zijn.)

Minor change (typo, formatting, version bump)
Nieuwe feature
Verbetering oude feature
Bugfix
Refactor
Aanpassing van de configuratie
Breaking change

Checklist:

Ik heb de code in deze PR zelf nogmaals nagekeken
Ik heb mijn code beter achtergelaten dan dat ik het aantrof
De code is leesbaar en de moeilijke onderdelen zijn voorzien van commentaar
Ik heb de tests toegevoegd/uitgebreid indien nodig
Ik heb de tests gedraaid die de werking van mijn wijziging bewijst
De PDOK documentatie is bijgewerkt indien nodig.
Er zit geen gevoelig informatie in deze PR (wachtwoorden etc)


          Update linting rules as discussed

bbf1758

roelarents approved these changes

View reviewed changes


          Fix linting issues, and suppress a few for later review

849108f

roelarents approved these changes

View reviewed changes

rkettelerij merged commit 5dcd619 into master

2 checks passed

rkettelerij deleted the linting branch

November 22, 2023 14:36

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet