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.

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

Remove unused script #1574

Closed

Conversation

gravitystorm
Copy link
Collaborator

Refs #309, where the comments shows that it hasn't been used "for
several years", and that comment was several years ago.

Refs openstreetmap#309, where the comments shows that it hasn't been used "for
several years", and that comment was several years ago.
@tomhughes
Copy link
Member

Well it hasn't been used, but it probably should have been... Of course script/locale/update-languages would need to work so we could update the yaml file, and that currently fails because the page it fetches from at Translatewiki no longer exists.

I have to say my memory was that the language list had come from wikipedia, or the ISO list or something and I don't remember getting it from Translatewiki at all!

Of course really we probably shouldn't have this in the database at all, though I think the diary entries have a foreign key against it.

There are several other bits of ancient cruft in script/locale that are even better candidates for cleanup than this for sure.

@gravitystorm
Copy link
Collaborator Author

There are several other bits of ancient cruft in script/locale that are even better candidates for cleanup than this for sure.

I'm sure there are! But I was just going on the discussion in the referenced PR.

@gravitystorm gravitystorm deleted the reload-languages branch May 9, 2018 03:24
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.

None yet

2 participants