Skip to content

Commit 09aa5c5

Browse files
authored
⬆️ Upgrade To Ember-Cli 3.10 (#49)
* ⬆️ v3.4.4...v3.5.1 * v3.5.0 * ⬆️ v3.5.1...v3.6.1 * v3.6.0 * ⬆️ v3.6.1...v3.7.1 * v3.7.0 * ⬆️ v3.7.1...v3.8.3 * v3.8.0 * ⬆️ v3.8.3...v3.9.0 * v3.9.0 * ⬆️ v3.9.0...v3.10.1 * v3.10.0
1 parent 9c4b49f commit 09aa5c5

18 files changed

+1060
-1457
lines changed

.eslintignore

+2
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,11 @@
88

99
# dependencies
1010
/bower_components/
11+
/node_modules/
1112

1213
# misc
1314
/coverage/
15+
!.*
1416

1517
# ember-try
1618
/.node_modules.ember-try/

.eslintrc.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
module.exports = {
22
root: true,
33
parserOptions: {
4-
ecmaVersion: 2017,
4+
ecmaVersion: 2018,
55
sourceType: 'module',
66
},
77
plugins: ['ember'],
@@ -14,6 +14,7 @@ module.exports = {
1414
// node files
1515
{
1616
files: [
17+
'.eslintrc.js',
1718
'.template-lintrc.js',
1819
'ember-cli-build.js',
1920
'index.js',
@@ -23,7 +24,6 @@ module.exports = {
2324
'tests/dummy/config/**/*.js',
2425
],
2526
excludedFiles: ['addon/**', 'addon-test-support/**', 'app/**', 'tests/dummy/app/**'],
26-
excludedFiles: ['app/**', 'addon/**', 'tests/dummy/app/**'],
2727
parserOptions: {
2828
sourceType: 'script',
2929
ecmaVersion: 2015,

.github/workflows/build.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ jobs:
1414
- name: Set Node.js
1515
uses: actions/setup-node@master
1616
with:
17-
node-version: 10.24.1
17+
node-version: 12.22.8
1818

1919
- name: Get yarn cache
2020
id: yarn-cache

.gitignore

+2
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@
99
/node_modules/
1010

1111
# misc
12+
/.env*
13+
/.pnp*
1214
/.sass-cache
1315
/connect.lock
1416
/coverage/

.npmignore

+4-1
Original file line numberDiff line numberDiff line change
@@ -9,13 +9,16 @@
99
/.bowerrc
1010
/.editorconfig
1111
/.ember-cli
12+
/.env*
1213
/.eslintignore
1314
/.eslintrc.js
1415
/.gitignore
15-
/.watchmanconfig
16+
/.template-lintrc.js
1617
/.travis.yml
18+
/.watchmanconfig
1719
/bower.json
1820
/config/ember-try.js
21+
/CONTRIBUTING.md
1922
/ember-cli-build.js
2023
/testem.js
2124
/tests/

.tool-versions

+1-1
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
nodejs 10.24.1
1+
nodejs 12.22.8

.travis.yml

+9-3
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ language: node_js
33
node_js:
44
# we recommend testing addons with the same minimum supported node version as Ember CLI
55
# so that your addon works for all apps
6-
- '6'
6+
- '8'
77

88
sudo: false
99
dist: trusty
@@ -19,6 +19,12 @@ env:
1919
# See https://git.io/vdao3 for details.
2020
- JOBS=1
2121

22+
branches:
23+
only:
24+
- master
25+
# npm version tags
26+
- /^v\d+\.\d+\.\d+/
27+
2228
jobs:
2329
fail_fast: true
2430
allow_failures:
@@ -43,8 +49,8 @@ jobs:
4349
# we recommend new addons test the current and previous LTS
4450
# as well as latest stable release (bonus points to beta/canary)
4551
- stage: 'Additional Tests'
46-
env: EMBER_TRY_SCENARIO=ember-lts-2.16
47-
- env: EMBER_TRY_SCENARIO=ember-lts-2.18
52+
env: EMBER_TRY_SCENARIO=ember-lts-2.18
53+
- env: EMBER_TRY_SCENARIO=ember-lts-3.4
4854
- env: EMBER_TRY_SCENARIO=ember-release
4955
- env: EMBER_TRY_SCENARIO=ember-beta
5056
- env: EMBER_TRY_SCENARIO=ember-canary

.vscode/settings.json

+5
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
{
2+
"yaml.schemas": {
3+
"https://json.schemastore.org/github-workflow.json": "file:///Users/dnelson/Code/ember-addons/ember-data-bootstrap3-forms/.github/workflows/build.yml"
4+
}
5+
}

CONTRIBUTING.md

+15-12
Original file line numberDiff line numberDiff line change
@@ -2,26 +2,27 @@
22

33
## Installation
44

5-
- `git clone <repository-url>`
6-
- `cd ember-cli-text-support-mixins`
7-
- `yarn install`
5+
```
6+
* `git clone <repository-url>`
7+
* `cd ember-data-bootstrap3-forms`
8+
* `yarn install`
89
910
## Linting
1011
11-
- `yarn lint:hbs`
12-
- `yarn lint:js`
13-
- `yarn lint:js --fix`
12+
* `yarn lint:hbs`
13+
* `yarn lint:js`
14+
* `yarn lint:js --fix`
1415
1516
## Running tests
1617
17-
- `ember test` – Runs the test suite on the current Ember version
18-
- `ember test --server` – Runs the test suite in "watch mode"
19-
- `ember try:each` – Runs the test suite against multiple Ember versions
18+
* `ember test` – Runs the test suite on the current Ember version
19+
* `ember test --server` – Runs the test suite in "watch mode"
20+
* `ember try:each` – Runs the test suite against multiple Ember versions
2021
2122
## Running the dummy application
2223
23-
- `ember serve`
24-
- Visit the dummy application at [http://localhost:4200](http://localhost:4200).
24+
* `ember serve`
25+
* Visit the dummy application at [http://localhost:4200](http://localhost:4200).
2526
2627
For more information on using ember-cli, visit [https://ember-cli.com/](https://ember-cli.com/).
2728
@@ -57,9 +58,11 @@ Make sure your `~/.aws/credentials` file has a profile named _cybertooth_
5758
with a valid key and secret,
5859
5960
```
61+
6062
[cybertooth]
6163
aws_access_key_id = <KEY>
6264
aws_secret_access_key = <SECRET>
65+
6366
```
6467
6568
Deploy by invoking the following command: `ember deploy production`
@@ -69,8 +72,8 @@ Confirm your changes are showing up in our S3 container: http://ember-data-boots
6972
# Releasing & Publishing To NPM
7073
7174
```
75+
7276
npm version x.y.z-sub.#
7377
git push
7478
git push --tags
7579
npm publish
76-
```

README.md

+8-37
Original file line numberDiff line numberDiff line change
@@ -6,20 +6,18 @@
66

77
This addon helps present your Ember-Data model errors in a consistent fashion using Bootstrap3 alerts and help-blocks.
88

9-
## Requirements
9+
## Compatibility
1010

11-
- Ember >= 1.13.0
12-
- Ember CLI
13-
- Ember Data - the `twbs-errors-alert` & `twbs-form-group` components depend on Ember Data's
11+
- Ember.js v2.18 or above
12+
- Ember CLI v2.13 or above
13+
- Node.js v8 or above
14+
15+
* Ember Data - the `twbs-errors-alert` & `twbs-form-group` components depend on Ember Data's
1416
modeling of errors.
15-
- You should have Bootstrap 3.x installed in your Ember application. Feel free to use the
16-
[ember-cli-bootstrap3-sass](http://emberobserver.com/addons/ember-cli-bootstrap3-sass)
17-
addon to setup Bootstrap if you haven't already done so.
17+
* You should have Bootstrap 3.x
1818

1919
## Installation
2020

21-
The following will install this addon:
22-
2321
```
2422
ember install ember-data-bootstrap3-forms
2523
```
@@ -122,36 +120,9 @@ component is watching that particular view of the errors.
122120
1. Do you have Ember Data in your application and are using it for your data modeling?
123121
1. Is Bootstrap 3 installed? As the addon's name states, this isn't Bootstrap 4 ready.
124122

125-
```
126-
ember install my-addon
127-
```
128-
129123
## Contributing
130124

131-
### Installation
132-
133-
- `git clone <repository-url>`
134-
- `cd my-addon`
135-
- `npm install`
136-
137-
### Linting
138-
139-
- `yarn lint:hbs`
140-
- `yarn lint:js`
141-
- `yarn lint:js --fix`
142-
143-
### Running tests
144-
145-
- `ember test` – Runs the test suite on the current Ember version
146-
- `ember test --server` – Runs the test suite in "watch mode"
147-
- `ember try:each` – Runs the test suite against multiple Ember versions
148-
149-
### Running the dummy application
150-
151-
- `ember serve`
152-
- Visit the dummy application at [http://localhost:4200](http://localhost:4200).
153-
154-
For more information on using ember-cli, visit [https://ember-cli.com/](https://ember-cli.com/).
125+
See the [Contributing](CONTRIBUTING.md) guide for details.
155126

156127
## License
157128

addon/components/twbs-form.js

+5-8
Original file line numberDiff line numberDiff line change
@@ -14,16 +14,13 @@ export default Component.extend({
1414
layout,
1515
tagName: 'form',
1616
didInsertElement() {
17-
const resetAction = this.reset;
18-
if (isPresent(resetAction)) {
19-
this.$()
20-
.off('reset.twbs-form')
21-
.on('reset.twbs-form', function () {
22-
resetAction();
23-
});
17+
if (isPresent(this.reset)) {
18+
this.element.addEventListener('reset', this.reset);
2419
}
2520
},
2621
willDestroyElement() {
27-
this.$().off('reset.twbs-form');
22+
if (isPresent(this.reset)) {
23+
this.element.removeEventListener('reset', this.reset);
24+
}
2825
},
2926
});

config/ember-try.js

+8-8
Original file line numberDiff line numberDiff line change
@@ -7,26 +7,22 @@ module.exports = function () {
77
return {
88
scenarios: [
99
{
10-
name: 'ember-lts-2.16',
10+
name: 'ember-lts-2.18',
1111
env: {
1212
EMBER_OPTIONAL_FEATURES: JSON.stringify({ 'jquery-integration': true }),
1313
},
1414
npm: {
1515
devDependencies: {
1616
'@ember/jquery': '^0.5.1',
17-
'ember-source': '~2.16.0',
17+
'ember-source': '~2.18.0',
1818
},
1919
},
2020
},
2121
{
22-
name: 'ember-lts-2.18',
23-
env: {
24-
EMBER_OPTIONAL_FEATURES: JSON.stringify({ 'jquery-integration': true }),
25-
},
22+
name: 'ember-lts-3.4',
2623
npm: {
2724
devDependencies: {
28-
'@ember/jquery': '^0.5.1',
29-
'ember-source': '~2.18.0',
25+
'ember-source': '~3.4.0',
3026
},
3127
},
3228
},
@@ -54,6 +50,10 @@ module.exports = function () {
5450
},
5551
},
5652
},
53+
// The default `.travis.yml` runs this scenario via `yarn test`,
54+
// not via `ember try`. It's still included here so that running
55+
// `ember try:each` manually or from a customized CI config will run it
56+
// along with all the other scenarios.
5757
{
5858
name: 'ember-default',
5959
npm: {

package.json

+15-16
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "ember-data-bootstrap3-forms",
3-
"version": "3.4.0",
3+
"version": "3.10.0",
44
"description": "The default blueprint for ember-cli addons.",
55
"keywords": [
66
"ember-addon"
@@ -40,43 +40,42 @@
4040
"update-browserslist": "npx browserslist@latest --update-db"
4141
},
4242
"dependencies": {
43-
"ember-cli-babel": "^6.16.0",
44-
"ember-cli-htmlbars": "^3.0.0"
43+
"ember-cli-babel": "^7.7.3",
44+
"ember-cli-htmlbars": "^3.0.1"
4545
},
4646
"devDependencies": {
4747
"@ember/jquery": "^1.1",
48-
"@ember/optional-features": "^0.6.3",
48+
"@ember/optional-features": "^0.7.0",
4949
"bootstrap": "3.4.1",
5050
"bootstrap-sass": "3.4.1",
5151
"bootswatch": "^3.3.7",
52-
"broccoli-asset-rev": "^2.7.0",
53-
"ember-ajax": "^5.1.1",
54-
"ember-cli": "~3.4.4",
55-
"ember-cli-dependency-checker": "^3.0.0",
52+
"broccoli-asset-rev": "^3.0.0",
53+
"ember-cli": "~3.10.1",
54+
"ember-cli-dependency-checker": "^3.1.0",
5655
"ember-cli-deploy": "^1.0.1",
5756
"ember-cli-deploy-build": "^1.1.0",
5857
"ember-cli-deploy-gzip": "^2.0.1",
5958
"ember-cli-deploy-s3": "^1.1.0",
60-
"ember-cli-eslint": "^4.2.3",
61-
"ember-cli-htmlbars-inline-precompile": "^1.0.3",
59+
"ember-cli-eslint": "^5.1.0",
60+
"ember-cli-htmlbars-inline-precompile": "^2.1.0",
6261
"ember-cli-inject-live-reload": "^2.1.0",
63-
"ember-cli-qunit": "^4.3.2",
6462
"ember-cli-sass": "^8.0.1",
6563
"ember-cli-sri": "^2.1.1",
6664
"ember-cli-template-lint": "^1.0.0-beta.1",
6765
"ember-cli-uglify": "^3.0.0",
6866
"ember-cli-update": "^0.56",
69-
"ember-data": "2.14",
67+
"ember-data": "~3.10.0",
7068
"ember-disable-prototype-extensions": "^1.1.3",
7169
"ember-export-application-global": "^2.0.0",
72-
"ember-load-initializers": "^1.1.0",
70+
"ember-load-initializers": "^2.0.0",
7371
"ember-maybe-import-regenerator": "^0.1.6",
72+
"ember-qunit": "^4.4.1",
7473
"ember-resolver": "^5.0.1",
7574
"ember-route-action-helper": "^2.0.6",
76-
"ember-source": "~3.4.0",
75+
"ember-source": "~3.10.0",
7776
"ember-source-channel-url": "^3.0.0",
7877
"ember-try": "^1.0.0",
79-
"eslint-plugin-ember": "^5.2.0",
78+
"eslint-plugin-ember": "^6.2.0",
8079
"eslint-plugin-node": "^11.1.0",
8180
"font-awesome": "4.7",
8281
"loader.js": "^4.7.0",
@@ -85,7 +84,7 @@
8584
"sass": "~1.14.3"
8685
},
8786
"engines": {
88-
"node": "6.* || 8.* || >= 10.*"
87+
"node": "8.* || >= 10.*"
8988
},
9089
"ember-addon": {
9190
"configPath": "tests/dummy/config"

0 commit comments

Comments
 (0)