Skip to content

docs: Add removal of dryRun to migration guide#397

Merged
lforst merged 1 commit intomainfrom
lforst-dryrun-migration-guide
Sep 5, 2023
Merged

docs: Add removal of dryRun to migration guide#397
lforst merged 1 commit intomainfrom
lforst-dryrun-migration-guide

Conversation

@lforst
Copy link
Copy Markdown

@lforst lforst commented Sep 5, 2023

Fixes #396

@lforst lforst requested review from Lms24 and mydea September 5, 2023 08:20
@lforst lforst changed the title docs: Add removal of dryRun to migratino guide docs: Add removal of dryRun to migration guide Sep 5, 2023
@lforst lforst merged commit c5dbca1 into main Sep 5, 2023
@lforst lforst deleted the lforst-dryrun-migration-guide branch September 5, 2023 08:50
@umanghome
Copy link
Copy Markdown

@lforst @Lms24 Is there an alternative to dryRun that is supported now?

@lforst
Copy link
Copy Markdown
Author

lforst commented Aug 27, 2024

No, running the plugin is generally non-destructive so running it is always safe, so we removed the option because it added incredible complexity.

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.

Fails with token error with dryRun enabled

3 participants