Skip to content

Tweak run script#72

Merged
hari-kuriakose merged 6 commits into
mainfrom
run-script-tweaks
Mar 10, 2024
Merged

Tweak run script#72
hari-kuriakose merged 6 commits into
mainfrom
run-script-tweaks

Conversation

@hari-kuriakose

@hari-kuriakose hari-kuriakose commented Mar 7, 2024

Copy link
Copy Markdown
Contributor

What

  • Few tweaks in run script - includes using latest tag by default
  • Add database migrations and env config to PR template
  • Update README
  • Update backend service dependencies in docker compose
  • Fix user ownership of build context

Notes on Testing

  • Verified that build is successful

Checklist

I have read and understood the Contribution Guidelines.

@jaseemjaskp jaseemjaskp left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hari-kuriakose
hari-kuriakose merged commit 8004f1a into main Mar 10, 2024
@hari-kuriakose
hari-kuriakose deleted the run-script-tweaks branch March 10, 2024 15:36
pk-zipstack pushed a commit that referenced this pull request Aug 20, 2025
* Add tweaks to run script
* Update readme
* Fix user ownership of build context
* Add db migrations and env changes to pr template

---------

Co-authored-by: Neha <115609453+nehabagdia@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants