Skip to content

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

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
Merged
luketomlinson merged 1 commit into from
Jun 10, 2021
Merged

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

luketomlinson merged 1 commit into from
Jun 10, 2021

Conversation

@C0ZEN
Copy link
Contributor

@C0ZEN C0ZEN commented Jun 8, 2021

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
luketomlinson
luketomlinson approved these changes Jun 10, 2021
View reviewed changes
Copy link
Collaborator

@luketomlinson luketomlinson left a comment

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