Skip to content

Conversation

@danadoherty639
Copy link
Contributor

Summary

Provide a detailed description of all the changes present in this pull request.

Additional Context

Add any additional context about the problem here.

  • Root cause and the steps to reproduce. (If applicable)
  • Thought process behind the implementation.

Related Issues (if any)

Mention any related issues or pull requests.

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.
  • Manually verified.

@danadoherty639 danadoherty639 requested review from a team as code owners October 16, 2024 12:04
@danadoherty639 danadoherty639 force-pushed the cat_2092 branch 2 times, most recently from be53cc7 to a84164d Compare October 16, 2024 14:04
Copy link
Contributor

@jordanbreen28 jordanbreen28 left a comment

Choose a reason for hiding this comment

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

Couple of comments @danadoherty639 - going to change label to maintenance as its CI work and doesnt need to appear in the changelog :)

bundle exec rake spec
on-failure-workflow-restarter-proxy:
# (1) run this job after the "acceptance" job and...
Copy link
Contributor

Choose a reason for hiding this comment

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

both comments need updating as refer to acceptance job which is a bit misleading

run: |
bundle exec rake spec
on-failure-workflow-restarter-proxy:
Copy link
Contributor

Choose a reason for hiding this comment

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

not sure we should be adding this to ci.yml, and only in nightly.yml like other repos, what do you think?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Completely agree, I'm just doing this for testing purposes, maybe its best I change it to a draft PR for now as still have a few tweaks to make to the workflow-restarter action. Thanks Jordan :)

@jordanbreen28 jordanbreen28 added maintenance Maintenance (PDK sync and other maintaining commits) and removed enhancement labels Oct 16, 2024
@danadoherty639 danadoherty639 marked this pull request as draft October 16, 2024 14:26
@danadoherty639 danadoherty639 force-pushed the cat_2092 branch 10 times, most recently from c075cca to 4b8bce6 Compare October 21, 2024 13:18
@danadoherty639 danadoherty639 marked this pull request as ready for review October 21, 2024 13:57
Copy link
Contributor

@gavindidrichsen gavindidrichsen left a comment

Choose a reason for hiding this comment

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

Great work @danadoherty639

@gavindidrichsen gavindidrichsen merged commit 4176a35 into main Oct 21, 2024
8 checks passed
@gavindidrichsen gavindidrichsen deleted the cat_2092 branch October 21, 2024 14:06
@danadoherty639 danadoherty639 restored the cat_2092 branch October 22, 2024 10:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Maintenance (PDK sync and other maintaining commits)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants