OdyseeTeam / chainquery

Chainquery parses and syncs the LBRY blockchain data into structured SQL
https://lbry.tech
MIT License
2.26k stars 42 forks source link

Fix for Dockerfile smell DL3048 #182

Open grosa1 opened 1 year ago

grosa1 commented 1 year ago

Hi! The Dockerfile placed at "docker/Dockerfile" contains the best practice violation DL3048 detected by the hadolint tool.

The smell DL3048 occurs when the pattern used for the label keys does not match the format recommended by official guidelines. In this pull request, we propose a fix for that smell generated by our fixing tool. We manually checked that the patch is correct before opening the pull request. To fix this smell, specifically, the label keys are refactored to match the correct format.

This change is only aimed at fixing that specific smell. In the case the fix is not valid or useful, please briefly indicate the reason along with suggestions for possible improvements.

Thanks in advance.