Skip to content

Comments

Fix remote deployment status text on update#1960

Merged
ronidelacruz-astro merged 2 commits intomainfrom
fix-remote-deployment-status-text
Oct 13, 2025
Merged

Fix remote deployment status text on update#1960
ronidelacruz-astro merged 2 commits intomainfrom
fix-remote-deployment-status-text

Conversation

@ronidelacruz-astro
Copy link
Contributor

Description

It fixes remote execution status text on deployment update

🎟 Issue(s)

Related #1959

  • Rebased from the main (or release if patching) branch (before testing)
  • Ran make test before taking out of draft
  • Ran make lint before taking out of draft
  • Added/updated applicable tests
  • Tested against Astro-API (if necessary).
  • Tested against Houston-API and Astronomer (if necessary).
  • Communicated to/tagged owners of respective clients potentially impacted by these changes.
  • Updated any related documentation

@ronidelacruz-astro ronidelacruz-astro merged commit 5fb1e4d into main Oct 13, 2025
4 of 5 checks passed
@ronidelacruz-astro ronidelacruz-astro deleted the fix-remote-deployment-status-text branch October 13, 2025 17:34
@coveralls-official
Copy link

Pull Request Test Coverage Report for Build d407a9e5-20f0-4f4e-b41b-01f6a23128e1

Details

  • 2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.001%) to 38.519%

Totals Coverage Status
Change from base Build 46e12efa-cbc7-4df4-a5bb-49449af7f41c: 0.001%
Covered Lines: 24140
Relevant Lines: 62670

💛 - Coveralls

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