-
Notifications
You must be signed in to change notification settings - Fork 90
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
Convert to v2 addon #576
Merged
Merged
Convert to v2 addon #576
Changes from 5 commits
Commits
Show all changes
6 commits
Select commit
Hold shift + click to select a range
e0bb3c3
Convert to monorepo
SergeAstapov 1a1c910
Convert to v2 addon
SergeAstapov 3ccd74b
docs: fix addon name
ro0gr e2fcb05
fix api docs generation
ro0gr cc29865
Merge pull request #2 from ro0gr/docs/v2-addon-fixups
SergeAstapov 440f438
Pull addon version from addon/package.json file
SergeAstapov File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -4,7 +4,6 @@ | |
|
||
root = true | ||
|
||
|
||
[*] | ||
end_of_line = lf | ||
charset = utf-8 | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,34 +1,28 @@ | ||
# See http://help.github.com/ignore-files/ for more about ignoring files. | ||
# See https://help.github.com/ignore-files/ for more about ignoring files. | ||
|
||
# compiled output | ||
/dist/ | ||
/tmp/ | ||
/tmp_docs | ||
dist/ | ||
tmp_docs/ | ||
|
||
# dependencies | ||
/bower_components/ | ||
/node_modules/ | ||
yarn.lock | ||
node_modules/ | ||
|
||
# *.md copies from release | ||
/addon/README.md | ||
/addon/LICENSE.md | ||
|
||
# misc | ||
/.sass-cache | ||
/.eslintcache | ||
/connect.lock | ||
/coverage/ | ||
/libpeerconnection.log | ||
.env* | ||
.eslintcache | ||
.pnp* | ||
.sass-cache | ||
connect.lock | ||
coverage/ | ||
libpeerconnection.log | ||
npm-debug.log* | ||
testem.log | ||
LICENSE.md | ||
/docs | ||
/_site | ||
/yarn-error.log | ||
jsconfig.json | ||
yarn-error.log | ||
.DS_Store | ||
|
||
# IDE/editor files | ||
/.idea/ | ||
/*.iml | ||
|
||
# ember-try | ||
/.node_modules.ember-try/ | ||
/bower.json.ember-try | ||
/package.json.ember-try | ||
/*.iml |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,26 +1,37 @@ | ||
# How To Contribute | ||
|
||
This repo is divided into multiple packages using npm workspaces: | ||
|
||
- `addon` is the actual ember-cli-page-object addon | ||
- `test-app` is its test suite | ||
- `docs` is the documentation/marketing site | ||
|
||
## Installation | ||
|
||
* `git clone <repository-url>` | ||
* `cd my-addon` | ||
* `git clone https:/san650/ember-cli-page-object.git` | ||
* `cd ember-cli-page-object` | ||
* `npm install` | ||
|
||
## Linting | ||
|
||
* `npm run lint:hbs` | ||
* `npm run lint:js` | ||
* `npm run lint:js -- --fix` | ||
Inside any of the packages you can run: | ||
|
||
* `npm run lint` | ||
* `npm run lint:fix` | ||
|
||
## Running tests | ||
|
||
* `ember test` – Runs the test suite on the current Ember version | ||
* `ember test --server` – Runs the test suite in "watch mode" | ||
* `ember try:each` – Runs the test suite against multiple Ember versions | ||
* `cd addon && npm run start` – Builds the addon in "watch mode" so changes picked up by test app. | ||
* `cd test-app && ember test` – Runs the test suite on the current Ember version | ||
* `cd test-app && ember test --server` – Runs the test suite in "watch mode" | ||
* `cd test-app && ember try:each` – Runs the test suite against multiple Ember versions | ||
|
||
During development, if you'd like test app to pick up changes in the addon, make sure to run both | ||
`cd addon && npm run start` and `cd test-app && ember test --server` in different terminals. | ||
|
||
## Running the dummy application | ||
## Running the test application | ||
|
||
* `ember serve` | ||
* Visit the dummy application at [http://localhost:4200](http://localhost:4200). | ||
* `cd test-app && ember serve` | ||
* Visit the test application at [http://localhost:4200](http://localhost:4200). | ||
|
||
For more information on using ember-cli, visit [https://ember-cli.com/](https://ember-cli.com/). |
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
# Ember Page Objects | ||
# Ember CLI Page Object | ||
|
||
[![Build Status](https://travis-ci.org/san650/ember-cli-page-object.svg?branch=master)](https://travis-ci.org/san650/ember-cli-page-object) | ||
[![Ember Observer Score](http://emberobserver.com/badges/ember-cli-page-object.svg)](http://emberobserver.com/addons/ember-cli-page-object) | ||
|
@@ -24,9 +24,12 @@ You can find more information about this design pattern here: | |
* [Page Objects - Selenium wiki](https:/SeleniumHQ/selenium/wiki/PageObjects) | ||
* [PageObject - Martin Fowler](http://martinfowler.com/bliki/PageObject.html) | ||
|
||
Compatibility | ||
------------------------------------------------------------------------------ | ||
|
||
* Ember.js v3.16 or above | ||
* Ember CLI v2.13 or above | ||
* Node.js v10 or above | ||
* Node.js v12 or above | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
|
||
|
||
Let's work together to improve this addon! | ||
|
||
|
@@ -45,11 +48,13 @@ Or you can install the NPM package directly. | |
$ npm install --save-dev ember-cli-page-object | ||
``` | ||
|
||
## Documentation | ||
Documentation | ||
------------------------------------------------------------------------------ | ||
|
||
Check the [site](http://ember-cli-page-object.js.org/) for full documentation. | ||
|
||
## Blueprints | ||
Blueprints | ||
------------------------------------------------------------------------------ | ||
|
||
The addon includes the following blueprints | ||
|
||
|
@@ -66,27 +71,10 @@ installing | |
create tests/pages/users.js | ||
``` | ||
|
||
## Development | ||
|
||
### Installation | ||
|
||
```sh | ||
$ git clone https:/san650/ember-cli-page-object.git | ||
$ cd $_ | ||
$ yarn install # or npm install | ||
``` | ||
|
||
### Linting | ||
|
||
* `npm run lint:hbs` | ||
* `npm run lint:js` | ||
* `npm run lint:js -- --fix` | ||
|
||
### Running Tests | ||
Contributing | ||
------------------------------------------------------------------------------ | ||
|
||
* `ember test` – Runs the test suite on the current Ember version | ||
* `ember test --server` – Runs the test suite in "watch mode" | ||
* `ember try:each` – Runs the test suite against multiple Ember versions | ||
See the [Contributing](CONTRIBUTING.md) guide for details. | ||
|
||
### Project's health | ||
|
||
|
@@ -110,4 +98,4 @@ $ yarn install # or npm install | |
|
||
ember-cli-page-object is licensed under the MIT license. | ||
|
||
See [LICENSE](./LICENSE) for the full license text. | ||
See [LICENSE](./LICENSE.md) for the full license text. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,15 @@ | ||
# unconventional js | ||
/blueprints/*/files/ | ||
|
||
# compiled output | ||
/dist/ | ||
/tmp/ | ||
|
||
# dependencies | ||
/node_modules/ | ||
|
||
# misc | ||
/coverage/ | ||
!.* | ||
.*/ | ||
.eslintcache |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
npm@8
is needed for workspaces support