Skip to content

Commit

Permalink
chore: update traivs config
Browse files Browse the repository at this point in the history
- use travis wait
- build without embroider since currently not supported
- use single travis step instead of one for each try step
- add info about why builds are commented out
- comment out lint task
- remove branches from travis config
  • Loading branch information
frederickfogerty committed Dec 9, 2021
1 parent 2e25379 commit f7bd774
Show file tree
Hide file tree
Showing 2 changed files with 40 additions and 53 deletions.
28 changes: 3 additions & 25 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,40 +16,18 @@ env:
- JOBS=1
- secure: jdfwWkawHI6Pxbkq64ihtdu0NnQbvLQrZXpaGDaavOn03ZA5sj1JCMKaw46Fg/q84sQl0VPUWDVXKBiGDvYTWFI84h3UZEtLZl6I0x6j5sV/6g1zzLI6AZ/v2VE/ELRytTTQ4FdPcKlxi9Gn5v7vi3lpWYvMaU8IAWVoZvSPj0R7jwolR44/O2nLzcqH98Dzw5LKMPT9PMgoz7f4C/8YewjVtZgHzQXEZkipkWHDmt9LzulU4Im5s7UJGPPGK/BWBDgxrpScetbxO5hOZS1dFZz0v5WBAwWxYRd1tD+SCXkKNHrJi7+QMnAw68NxyGzDRN7YD+AWNMhjPEGt2fXUkX+sBRTOUoJ+BuCsVBpiROvN3glv8WUc0IaYPLS4xnalN1/e4ArHXVgLYockYTNlOpo4Oo0S3R2u4reYvQxCjTVaFERXRm1Q+RQe+3sfLPmO/uswMtkIGZ7n9Xd8ER3y+z0cxZS2S5aghF3lmATsmrU3CP8HvkRsr9fy/DDGNr2BEvoK64oGc7kHdc4EqPOauPYKZNXH4DdAaSpTmDmYGG5bkkAQ+pRE+h9VhBd5xvZMQmG6aH+xSIvpOCkBzF5U8OVJr0/qhWg9BzUl6kgua+Ri9MDFcwqKv9bR20kYv3krDSkV4zYvvG0fDbllrW76fDF7l3I4ysGWxspLfik2qu0=

branches:
only:
- master
# npm version tags
- /^v\d+\.\d+\.\d+/

jobs:
fast_finish: true
allow_failures:
- env: EMBER_TRY_SCENARIO=ember-canary

include:
# runs linting and tests with current locked deps
- stage: "Tests"
name: "Tests"
script:
- yarn lint
# TODO: fix in later PR
# - yarn lint
- yarn test:ember
- travis_wait yarn test:ember-compatibility

- stage: "Additional Tests"
name: "Floating Dependencies"
install:
- yarn install --no-lockfile --non-interactive
script:
- yarn test:ember-compatibility

# we recommend new addons test the current and previous LTS
# as well as latest stable release (bonus points to beta/canary)
- env: EMBER_TRY_SCENARIO=ember-lts-3.20
- env: EMBER_TRY_SCENARIO=ember-lts-3.24
- env: EMBER_TRY_SCENARIO=ember-lts-3.28
# - env: EMBER_TRY_SCENARIO=ember-release
# - env: EMBER_TRY_SCENARIO=ember-beta
# - env: EMBER_TRY_SCENARIO=ember-canary
- env: EMBER_TRY_SCENARIO=ember-classic
# - env: EMBER_TRY_SCENARIO=embroider-safe
# - env: EMBER_TRY_SCENARIO=embroider-optimized
65 changes: 37 additions & 28 deletions config/ember-try.js
Original file line number Diff line number Diff line change
Expand Up @@ -31,32 +31,6 @@ module.exports = async function () {
},
},
},
// Comment out the release, beta and canary builds as they use Ember@4
// which is not compatible atm
// {
// name: 'ember-release',
// npm: {
// devDependencies: {
// 'ember-source': await getChannelURL('release'),
// },
// },
// },
// {
// name: 'ember-beta',
// npm: {
// devDependencies: {
// 'ember-source': await getChannelURL('beta'),
// },
// },
// },
// {
// name: 'ember-canary',
// npm: {
// devDependencies: {
// 'ember-source': await getChannelURL('canary'),
// },
// },
// },
{
name: 'ember-classic',
env: {
Expand All @@ -75,9 +49,44 @@ module.exports = async function () {
},
},
},
// The rest of the scenarios are allowed to fail as they use Ember@4,
// which is not supported by this library yet
{
name: 'ember-release',
allowedToFail: true,
npm: {
devDependencies: {
'ember-source': await getChannelURL('release'),
},
},
},
{
name: 'ember-beta',
allowedToFail: true,
npm: {
devDependencies: {
'ember-source': await getChannelURL('beta'),
},
},
},
{
name: 'ember-canary',
allowedToFail: true,
npm: {
devDependencies: {
'ember-source': await getChannelURL('canary'),
},
},
},
// Remove Embroider builds for now as there are compat issues
// embroiderSafe(),
// embroiderOptimized(),
// {
// ...embroiderSafe(),
// allowedToFail: true,
// },
// {
// ...embroiderOptimized(),
// allowedToFail: true,
// }
],
};
};

0 comments on commit f7bd774

Please sign in to comment.