Skip to content

Update Josm_FranceSpecificRules.py for add syntax check ref:FR:NAF #2524

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

Closed
wants to merge 1 commit into from

Conversation

cayenne17
Copy link

#2523
fix #2523

There are things to adapt and modify

@frodrigo
Copy link
Contributor

You made this change manually ? This file is generated from JOSM MapCSS file.

@cayenne17
Copy link
Author

cayenne17 commented Jun 18, 2025

You made this change manually ?

Yes. I don't know if this is the way to do it?

@Famlam
Copy link
Collaborator

Famlam commented Jun 18, 2025

It's a synchronisation performed by a script (mapcss/update.py) that updates all external rules at once.

I see you already updated the external mapcss rules. I can trigger the script if you wish? (I need to make a modification to an exception for the Dutch rules anyway)

@cayenne17
Copy link
Author

I can trigger the script if you wish?

@Famlam : I'll leave you to it.

@Famlam
Copy link
Collaborator

Famlam commented Jun 18, 2025

See #2525. I've added a message to the generated python files as well to make sure people know it's automatically generated.

@Famlam Famlam closed this Jun 18, 2025
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.

Add ref:FR:NAF syntax check
3 participants