-
Notifications
You must be signed in to change notification settings - Fork 6
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
Testing with ember-cli-page-object v2.0.0-beta.3 #36
Conversation
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'd suggest that we merge this to master
and release it as an 0.8.0-beta
for the ember-table 4.0-beta
branch to use? Then when ember-cli-page-object releases a 2.0 stable, we can upgrade to that, release an 0.8
stable here, and then leverage that in the new version of ember-table?
It looks like @mixonic and @twokul have NPM publish permissions for this addon, so hopefully one of them can review/merge this and get a release cut?
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.
One small comment, but lgtm overally!
.github/workflows/ci.yml
Outdated
@@ -20,7 +20,7 @@ jobs: | |||
- name: Node Modules Cache | |||
uses: actions/cache@v2 | |||
with: | |||
path: '**/node_modules' | |||
path: "**/node_modules" |
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.
It seems we use single quotes in this file. Why the change?
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.
reverted.
Going to publish beta today |
This PR is intended our efforts to get
ember-table
compatible with Ember v4.Changes include:
v2.0.0-beta3
branch ofember-cli-page-object
https://github.com/san650/ember-cli-page-object/releasesuseNativeEvents
Perhaps we could release a beta version of this addon while we wait for a release version of v2 of
ember-cli-page-object