marquez icon indicating copy to clipboard operation
marquez copied to clipboard

Improve readability for the up.sh script

Open rafafrdz opened this issue 11 months ago • 2 comments

Problem

The up.sh script was functional but not very readable. This change is aimed at improving its clarity and maintainability.

Solution

Refactored the up.sh script to make the use of environment variables more explicit and easier to follow. This is a readability improvement only, no change in functionality or behavior.

One-line summary: Improve readability of up.sh by making environment variable usage more explicit.

Checklist

  • [x] You've signed-off your work
  • [ ] Your changes are accompanied by tests (if relevant)
  • [x] Your change contains a small diff and is self-contained
  • [ ] You've updated any relevant documentation (if relevant)
  • [x] You've included a one-line summary of your change for the CHANGELOG.md (Depending on the change, this may not be necessary).
  • [ ] You've versioned your .sql database schema migration according to Flyway's naming convention (if relevant)
  • [ ] You've included a header in any source code files (if relevant)

rafafrdz avatar May 16 '25 12:05 rafafrdz

Thanks for opening your first pull request in the Marquez project! Please check out our contributing guidelines (https://github.com/MarquezProject/marquez/blob/main/CONTRIBUTING.md).

boring-cyborg[bot] avatar May 16 '25 12:05 boring-cyborg[bot]

Deploy Preview for peppy-sprite-186812 failed.

Name Link
Latest commit 9b08b0f2bce03bfe476fec0700caebcc9dbad1e8
Latest deploy log https://app.netlify.com/projects/peppy-sprite-186812/deploys/682728fe702af3000869b6df

netlify[bot] avatar May 16 '25 12:05 netlify[bot]