-
Notifications
You must be signed in to change notification settings - Fork 7
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
Feature/cypress test #148
base: develop
Are you sure you want to change the base?
Feature/cypress test #148
Conversation
pull_request: | ||
branches: | ||
- develop | ||
jobs: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for working on this @sudip-md
For the pipeline, we run the tests on the release ZIP. You can refer to how we're doing it in other plugins such as Restricted Site Access and Simple Local Avatars.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Sidsector9 probably best if you can help update this and get merged in
Description of the Change
I have added test cases for Ad-Refresh-Control plugin and it is working fine in local.
Where possible, please also include:
-->
How to test the Change
Please setup plugin in loca
Checklist: