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.

Sign up for GitHub

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

Jump to bottom

[no-Jira] Add retry logic to GraphQL Codegen action #880

Merged
merged 1 commit into from
Feb 23, 2024
Merged

[no-Jira] Add retry logic to GraphQL Codegen action #880

merged 1 commit into from
Feb 23, 2024

Conversation

canac
Copy link
Contributor

@canac canac commented Feb 21, 2024 •
edited
Loading

Description

  • Retry gql codegen commands in GitHub Actions because they fail often

Checklist:

  • I have given my PR a title with the format "MPDX-(JIRA#) (summary sentence max 80 chars)"
  • I have applied the appropriate labels. (Add the label "On Staging" to get the branch automatically merged into staging.)
  • I have requested a review from another person on the project

@aws-amplify-us-east-1 AWS Amplify (us-east-1)
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-880.d3dytjb8adxkk5.amplifyapp.com

@canac canac requested a review from dr-bizz February 21, 2024 22:53
@canac canac added the On Staging Will be merged to the staging branch by Github Actions label Feb 21, 2024
dr-bizz
dr-bizz approved these changes Feb 23, 2024
View reviewed changes
Copy link
Contributor

@dr-bizz dr-bizz left a comment

Choose a reason for hiding this comment

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

It looks good. Great work finding this.

@canac canac merged commit 5045624 into main Feb 23, 2024
18 checks passed
@canac canac deleted the retry-codegen branch February 23, 2024 14:13
@canac canac mentioned this pull request Feb 23, 2024
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dr-bizz dr-bizz dr-bizz approved these changes

Assignees
No one assigned
Labels
On Staging Will be merged to the staging branch by Github Actions
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@canac @dr-bizz