Skip to content

Conversation

@rafafrdz
Copy link

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

  • You've signed-off your work
  • Your changes are accompanied by tests (if relevant)
  • Your change contains a small diff and is self-contained
  • You've updated any relevant documentation (if relevant)
  • 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)

@boring-cyborg boring-cyborg bot added the docker label May 16, 2025
@boring-cyborg
Copy link

boring-cyborg bot commented May 16, 2025

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).

@netlify
Copy link

netlify bot commented May 16, 2025

Deploy Preview for peppy-sprite-186812 failed.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant