Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removed GAE workflows, build args from Dockerfile #403

Conversation

ericdrosas87
Copy link
Contributor

Removed GAE workflows, build args from Dockerfile

Removed GAE workflows, build args from Dockerfile
@ericdrosas87 ericdrosas87 requested a review from jtrouth March 11, 2024 17:56
@ericdrosas87
Copy link
Contributor Author

Resolves #401

Dockerfile Outdated
Comment on lines 25 to 27
ENV NEXT_PUBLIC_API_URL=$NEXT_PUBLIC_API_URL
ENV EDC_LOGGER_API_URL=$EDC_LOGGER_API_URL
ENV NEXT_PUBLIC_BASE_URL=$NEXT_PUBLIC_BASE_URL
Copy link
Contributor

Choose a reason for hiding this comment

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

These can be removed also

Removed build env var refs in `Dockerfile`
@ericdrosas87
Copy link
Contributor Author

Removed the remaining env vars associated with the build except for:

ARG RUN_BUILD="true"
ENV RUN_BUILD=${RUN_BUILD}

@jtrouth please confirm all looks good and then I'll merge

@ericdrosas87 ericdrosas87 merged commit 0a7f1c6 into develop Mar 11, 2024
2 checks passed
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.

2 participants