Skip to content

Conversation

@ZIJ
Copy link
Contributor

@ZIJ ZIJ commented Jan 31, 2024

Screenshot 2024-01-31 at 13 06 17

@ZIJ ZIJ requested a review from motatoes January 31, 2024 13:07
@ZIJ
Copy link
Contributor Author

ZIJ commented Jan 31, 2024

Confirmed that it still works after removing the duplicate plan run when stored plan does not exist

https://github.com/diggerhq/test-stored-plan/actions/runs/7728245021/job/21068578418

@ZIJ
Copy link
Contributor Author

ZIJ commented Jan 31, 2024

Confirmed working after addressing feedback: https://github.com/diggerhq/test-stored-plan/actions/runs/7729644664/job/21073306830

@ZIJ ZIJ merged commit 80a87ae into develop Jan 31, 2024
ben-of-codecraft pushed a commit to ben-of-codecraft/digger that referenced this pull request May 21, 2024
Use stored plan in pre-apply plan policy checks
breardon2011 pushed a commit that referenced this pull request Sep 17, 2025
Use stored plan in pre-apply plan policy checks
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.

3 participants