Skip to content
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.

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

feat: update plugin to be compatible with Cypress 10. #23

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

admah
Copy link

@admah admah commented Jul 26, 2022

BREAKING CHANGE: updates config for Cypress 10 and updates plugin to use es6 import/exports.

Closes #22

BREAKING CHANGE: updates config for Cypress 10 and updates plugin to use es6 import/exports.
@@ -1,6 +1,7 @@
{
"name": "cypress-log-to-output",
"version": "1.1.2",
"type": "module",
Copy link
Owner

Choose a reason for hiding this comment

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

is this change required for 10.0?

Copy link
Author

Choose a reason for hiding this comment

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

I think so. I added it so that I could use ES import instead of require.

Choose a reason for hiding this comment

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

Desperately need the v10 support, any chance this can be merged in?

Copy link

Choose a reason for hiding this comment

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

@flotwig Wanna maybe deprecate in favor of https://github.com/archfz/cypress-terminal-report ?

Choose a reason for hiding this comment

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

I also need the v10 support!

@kpturner
Copy link

I have found that the plugin still works OK (10.6.0) without any changes UNLESS you have a plugin after it that also adds a listener to before:browser:launch afterwards. Dunno if this is an issue with Cypress or not.

See cypress-io/cypress#23465

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.

Update plugin to be compatible with Cypress 10
7 participants