Skip to content

Modify workflow interfaces for describe call#64

Merged
chrisqm-dev merged 1 commit intomainfrom
refactor/stack-interface
Oct 13, 2025
Merged

Modify workflow interfaces for describe call#64
chrisqm-dev merged 1 commit intomainfrom
refactor/stack-interface

Conversation

@chrisqm-dev
Copy link
Contributor

@chrisqm-dev chrisqm-dev commented Oct 10, 2025

Description of changes:

  • Added describeStatus method to StackActionWorkflow
  • Added generics to StackActionWorkflow, so the describeStatus typing isn't static
  • Added logic for describeStatus
    • Validations are returned from describeStatus
    • Deployments are returned for describeStatus on deployment
    • Workflow failure reasons are returned for describeStatus

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@chrisqm-dev chrisqm-dev marked this pull request as ready for review October 10, 2025 20:13
@chrisqm-dev chrisqm-dev requested a review from a team as a code owner October 10, 2025 20:13
satyakigh
satyakigh previously approved these changes Oct 13, 2025
Zee2413
Zee2413 previously approved these changes Oct 13, 2025
Copy link
Contributor

@Zee2413 Zee2413 left a comment

Choose a reason for hiding this comment

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

approved prior to merge conflict resolution

@chrisqm-dev chrisqm-dev dismissed stale reviews from Zee2413 and satyakigh via 1ff9dbb October 13, 2025 19:31
@chrisqm-dev chrisqm-dev force-pushed the refactor/stack-interface branch from 109de23 to 1ff9dbb Compare October 13, 2025 19:31
@chrisqm-dev chrisqm-dev merged commit 73670e9 into main Oct 13, 2025
8 checks passed
@chrisqm-dev chrisqm-dev deleted the refactor/stack-interface branch October 21, 2025 20:38
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