Skip to content

🐛 Fix reference error when static config options are not defined#515

Merged
wwilsman merged 1 commit intomasterfrom
ww/fix-include-exclude-migration-error
Aug 20, 2021
Merged

🐛 Fix reference error when static config options are not defined#515
wwilsman merged 1 commit intomasterfrom
ww/fix-include-exclude-migration-error

Conversation

@wwilsman
Copy link
Copy Markdown
Contributor

What is this?

This PR adds a safety check to the migrate function to avoid errors when there are no static config options (which usually happens when using any other command).

More evidence that a root-level integration test suite is needed. This error only happens with other commands but each command is tested in isolation (mostly).

@wwilsman wwilsman added the 🐛 bug Something isn't working label Aug 20, 2021
@wwilsman wwilsman requested a review from Robdel12 August 20, 2021 20:01
Copy link
Copy Markdown
Contributor

@Robdel12 Robdel12 left a comment

Choose a reason for hiding this comment

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

🏁

@wwilsman wwilsman merged commit 860b80d into master Aug 20, 2021
@wwilsman wwilsman deleted the ww/fix-include-exclude-migration-error branch August 20, 2021 20:32
samarsault pushed a commit that referenced this pull request Mar 3, 2023
Bumps [cypress](https://github.com/cypress-io/cypress) from 10.1.0 to 10.2.0.
- [Release notes](https://github.com/cypress-io/cypress/releases)
- [Changelog](https://github.com/cypress-io/cypress/blob/develop/.releaserc.base.js)
- [Commits](cypress-io/cypress@v10.1.0...v10.2.0)

---
updated-dependencies:
- dependency-name: cypress
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants