-
Notifications
You must be signed in to change notification settings - Fork 10
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/playwright migration #78
Conversation
…cmfive-boilerplate into feature/PlaywrightMigration
…back add readme and gitignore
…cmfive-boilerplate into feature/PlaywrightMigration
…cmfive-boilerplate into feature/PlaywrightMigration
…ght test job doesn't fail whenever a module test fails
…cmfive-boilerplate into feature/PlaywrightMigration
.gitignore
Outdated
@@ -8,8 +8,6 @@ | |||
/storage/session | |||
/log | |||
|
|||
.phpunit.result.cache |
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.
Why do we version this now?
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.
We shouldn't not sure why this was removed
test/playwright/README.md
Outdated
@@ -0,0 +1,61 @@ | |||
# Get Cmfive Working | |||
|
|||
- Use this codespace: https://github.com/2pisoftware/codespace_dev_box/tree/BoilerplateCore_Modules_Tests_Debug |
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.
This now outdated? Also, per chatter elsewhere around doc's, this is not a cmfive public repo/reference?
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.
will remove the "Get Cmfive Working" part, was for Dane to follow while trying to get the newer codespace to be playwright friendly
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.
A couple of points raised, pedantic/confused?
Any response appreciated.
Found nothing to stop this going ahead.
Checklist
Description
Merging Playwright setup into develop