Closed breunigs closed 6 months ago
Introduced Dynamic Arguments
This PR modifies the Dockerfile associated with version 4.3 of the project to make it more adaptable. The key changes include the addition of NOMINATIM_VERSION
and USER_AGENT
as arguments. This will help in managing versions and user-agents more flexibly.
Elimination of Redundancy It also efficiently removes the repetitive definition of these variables from different lines of the codes. This reduces clutter making the Dockerfile more readable.
Enhanced Consistency
Furthermore, the PR consistently replaces the existing values of NOMINATIM_VERSION
and USER_AGENT
with these arguments in many places. This will ensure system-wide uniformity and reduce errors.
This change overall assists in simplifying the configuration process and enables more effective management of versions and user agents in the project.
Thanks for the PR.
@all-contributors please add @breunigs for code
@philipkozeny
I couldn't determine any contributions to add, did you specify any contributions? Please make sure to use valid contribution names.
I've put up a pull request to add @breunigs! :tada:
Testing:
cc @leonardehrenfried who initially introduced the comment
I couldn't find a policy/developer guide if all or only the most recent version should be patched. I can add the older ones if desired.