Skip to content

fix(dry-run): forbid mutations in dry-run - #500

Merged
luketomlinson merged 1 commit into
actions:mainfrom
C0ZEN:fix/dry-run
Jun 10, 2021
Merged

fix(dry-run): forbid mutations in dry-run#500
luketomlinson merged 1 commit into
actions:mainfrom
C0ZEN:fix/dry-run

Conversation

@C0ZEN

@C0ZEN C0ZEN commented Jun 8, 2021

Copy link
Copy Markdown
Contributor

Changes

@C0ZEN
fix(dry-run): forbid mutations in dry-run
4e719f7 
Bring back the dry-run by default for the tests - bad idea to disable it sorry
Fix bad documentation array format
Fixes actions#499
@C0ZEN
C0ZEN requested a review from a team as a code owner June 8, 2021 20:55
@C0ZEN C0ZEN mentioned this pull request Jun 9, 2021
Closed
3 tasks done
luketomlinson
luketomlinson approved these changes Jun 10, 2021
View reviewed changes

@luketomlinson luketomlinson left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

💥

@luketomlinson
luketomlinson merged commit f1017f3 into actions:main Jun 10, 2021
@C0ZEN
C0ZEN deleted the fix/dry-run branch June 10, 2021 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@luketomlinson luketomlinson luketomlinson approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Dry-run allow to add labels and update the issues :/

2 participants

@C0ZEN @luketomlinson