Skip to content

feat: add infra files#14

Merged
Joao-1 merged 2 commits intomainfrom
dev
Feb 11, 2026
Merged

feat: add infra files#14
Joao-1 merged 2 commits intomainfrom
dev

Conversation

@Joao-1
Copy link
Contributor

@Joao-1 Joao-1 commented Feb 11, 2026

No description provided.

Copilot AI review requested due to automatic review settings February 11, 2026 17:04
@Joao-1 Joao-1 merged commit 0e29e55 into main Feb 11, 2026
1 check passed
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Docker publish GitHub Actions workflow to use a different API base URL for builds originating from the main branch.

Changes:

  • Switch VITE_API_URL for main builds from backoffice-api.projetodesenvolve.dev to backoffice-api.projetodesenvolve.online.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 35 to 39
if [[ "${GITHUB_REF_NAME}" == "main" ]]; then
echo "VITE_API_URL=https://backoffice-api.projetodesenvolve.dev" >> "$GITHUB_ENV"
echo "VITE_API_URL=https://backoffice-api.projetodesenvolve.online" >> "$GITHUB_ENV"
else
echo "VITE_API_URL=https://backoffice-api.projetodesenvolve.dev" >> "$GITHUB_ENV"
fi
Copy link

Copilot AI Feb 11, 2026

Choose a reason for hiding this comment

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

The workflow runs on multiple branches (production/prd/homolog/hml/develop/dev) and tags (v*), but VITE_API_URL is only switched for main; all other refs (including production releases and tag builds) will be built against the .dev API. Consider mapping the URL based on GITHUB_REF_NAME (and github.ref_type/tag builds) so production and staging branches/tags point to the correct API endpoint(s).

Copilot uses AI. Check for mistakes.
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.

1 participant