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
GH action summary added #194
Conversation
It has been marked as ready for review though some Github action has failed. Failed ones are not related to this PR itself. |
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.
I've added a question inline.
I can confirm that argparse
is already required in the develop
branch and that's what looks to be triggering the dependency review action failure.
@@ -55,3 +55,22 @@ jobs: | |||
run: npm run env:start | |||
- name: Test | |||
run: npm run cypress:run | |||
- name: Update summary | |||
if: always() |
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.
I pushed this in ddf9924 to ensure the summary displays on failed test runs. See https://github.com/10up/insert-special-characters/actions/runs/4921439734 for an example.
package.json
Outdated
@@ -46,6 +46,7 @@ | |||
}, | |||
"dependencies": { | |||
"@wordpress/components": "^22.1.0", | |||
"cypress-mochawesome-reporter": "^3.4.0", |
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.
I think this should probably be in devDependencies for this repo alongside cypress. In this repo we include the react-character-map from the dependencies in the built version so there may be a risk we include this in the build unintentionally.
Are you able to double check?
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.
LGTM, thanks.
I'm not sure why the E2E tests are failing for some versions of WP. Possibly related to changes in the block editor markup.
Description of the Change
Mochawesome reporter added for Cypress test report
Closes #179
Changelog Entry
Credits
Props @jayedul
Checklist: