Skip to content

Betters the style of the Dockerfile

John Skarbek requested to merge jts/style into master

What does this MR do?

  • This removes a variable set inside of the shell script execute by Docker
  • This takes what used to be a variable and passes each item into the command where the variable was previously leveraged
  • Breaks up the contents of what used to be a variable and the item that was not part of the variable, and segregates them into an individual line and sorted for readability
  • While this doesn't change the outcome of the image, this changes the RUN line itself, which will force a change to the overall image

This change is purely aesthetic.

Related issues

n/a

Checklist

See Definition of done.

For anything in this list which will not be completed, please provide a reason in the MR discussion

Required

  • Merge Request Title, and Description are up to date, accurate, and descriptive
  • MR targeting the appropriate branch
  • MR has a green pipeline on GitLab.com

Expected (please provide an explanation if not completing)

  • Test plan indicating conditions for success has been posted and passes - n/a - no change to build output
  • Documentation created/updated - n/a - we do not have style guides for Dockerfiles
  • Integration tests added to GitLab QA - n/a - no change to build output
  • The impact any change in container size has should be evaluated
Edited by Jason Plum

Merge request reports