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

chore(workflows): set correct quotes on commands #6059

Merged
merged 1 commit into from
May 19, 2023

Conversation

levivannoort
Copy link
Member

@levivannoort levivannoort commented May 19, 2023

Close: #5975

PR Details

🤖 Generated by Copilot at 6a05b55

Summary

🐛🚀📝

Fix a typo in the production bucket name in the static deployment template. The change prevents deployment errors caused by invalid syntax.

A typo fixed
app.amplication.com now
Has its closing quote

Walkthrough

  • Fix typo in bucket name for production environment (link)

PR Checklist

  • Tests for the changes have been added
  • npm test doesn't throw any error

IMPORTANT: Please review the CONTRIBUTING.md file for detailed contributing guidelines.

@levivannoort levivannoort requested a review from overbit May 19, 2023 13:42
@levivannoort levivannoort self-assigned this May 19, 2023
@nx-cloud
Copy link

nx-cloud bot commented May 19, 2023

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 6a05b55. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch


✅ Successfully ran 4 targets

Sent with 💌 from NxCloud.

Copy link
Contributor

@overbit overbit left a comment

Choose a reason for hiding this comment

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

Your changes, a sight to behold,
Code transformed, shining gold.
Elegant logic, clarity refined,
Well done, your work, so divine!

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.

None yet

2 participants