Skip to content
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

Rename geojson_info.txt to geojson_info.overpassql #10

Closed
FloEdelmann opened this issue Dec 21, 2022 · 3 comments · Fixed by #11
Closed

Rename geojson_info.txt to geojson_info.overpassql #10

FloEdelmann opened this issue Dec 21, 2022 · 3 comments · Fixed by #11

Comments

@FloEdelmann
Copy link
Contributor

*.overpassql is now the recommended file extension for Overpass QL: OSM Wiki: Overpass QL § File extension
(Disclaimer: I added that section after some discussion.)
Also, it is the most widely used extension on GitHub, see https://github.com/users/FloEdelmann/projects/2/views/2

I propose to rename all Overpass QL files in this repo to *.overpassql for consistency.

Context: GitHub's syntax highlighting engine requires that 200 repositories use the same file extension before they consider supporting that syntax and file extension: github-linguist/linguist#5890
@FloEdelmann
Copy link
Contributor Author

@nefft0n Would you accept a pull request?

@nefft0n
Copy link
Collaborator

nefft0n commented Feb 15, 2023

Hey,

yes, of course.
Sorry, I didn't read your comment in the first place.

@FloEdelmann FloEdelmann mentioned this issue Feb 17, 2023
Merged
@FloEdelmann
Copy link
Contributor Author

FloEdelmann commented Feb 17, 2023 •
edited
Loading

@nefft0n I've created the PR: #11

@github-project-automation github-project-automation bot moved this to Using `*.overpassql` in Rename to `*.overpassql` Mar 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Rename geojson_info.txt to geojson_info.overpassql
2 participants
@FloEdelmann @nefft0n