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

docs: add cypress-intercept-formdata to plugins.json #5509

Merged
merged 3 commits into from
Oct 13, 2023

Conversation

yoavniran
Copy link
Contributor

@yoavniran yoavniran commented Sep 29, 2023

  • 🚀 Works with the latest major version of Cypress
  • 🛠 Plugin purpose is clearly documented (readme)
  • 📝 Well-written documentation (usage)
  • 🔬 Tested using Cypress - tests using Cypress can act as both example usage and test coverage (report)
  • 👷‍♀️ CI pipeline that's passing (GH actions)

@netlify
Copy link

netlify bot commented Sep 29, 2023

👷 Deploy request for cypress-docs pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit f8e8383

@netlify
Copy link

netlify bot commented Sep 29, 2023

Deploy Preview for benevolent-cat-040f48 ready!

Name Link
🔨 Latest commit f8e8383
🔍 Latest deploy log https://app.netlify.com/sites/benevolent-cat-040f48/deploys/651d3862f5484b0008c1f594
😎 Deploy Preview https://deploy-preview-5509--benevolent-cat-040f48.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@CLAassistant
Copy link

CLAassistant commented Sep 29, 2023

CLA assistant check
All committers have signed the CLA.

@cypress-app-bot
Copy link
Collaborator

src/data/plugins.json Outdated Show resolved Hide resolved
Co-authored-by: Mike McCready <[email protected]>
@yoavniran yoavniran requested a review from MikeMcC399 October 4, 2023 08:34
@yoavniran
Copy link
Contributor Author

hey @MikeMcC399
anything I need to do to help get this merged?
thanks

@MikeMcC399
Copy link
Contributor

Hi @yoavniran

anything I need to do to help get this merged? thanks

You could make sure that your plugin fits the checklist. Copy and paste this into your original post.

Before submitting your plugin, please check to see if it fulfills these requirements:

  • 🚀 Works with the latest major version of Cypress
  • 🛠 Plugin purpose is clearly documented (in a README or docs website)
  • 📝 Well-written documentation - A great example (link)
  • 🔬 Tested using Cypress - tests using Cypress can act as both example usage and test coverage
  • 👷‍♀️ CI pipeline that's passing (CircleCI and Github Actions are both free for Open Source)

If you have reason to believe your plugin does not need to meet certain requirements, please feel free to provide justification below:

Other than that, I'm only a community contributor. You would need to get approval from a Cypress member for merger. In fact I'm also waiting for several PRs, which I submitted, to be merged. It seems that Cypress changed their organization last week, so I don't know how this is going to be worked now.

@mschile
Copy link
Contributor

mschile commented Oct 13, 2023

Hi @yoavniran 👋, thanks for the plugin contribution!

@mschile mschile merged commit ca3ea49 into cypress-io:main Oct 13, 2023
3 checks passed
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.

5 participants