Skip to content
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

fix test runs, update lts support tests #8027

Merged
merged 10 commits into from
Jul 15, 2022
Merged

Conversation

runspired
Copy link
Contributor

No description provided.

@runspired runspired added 🎯 canary PR is targeting canary (default) 🏷️ chore This PR primarily refactors code or updates dependencies 🏷️ test This PR primarily adds tests for a feature labels Jul 15, 2022
@github-actions
Copy link

github-actions bot commented Jul 15, 2022

Asset Size Report for b43d5d0

Modern Builds

☑️ EmberData has not changed in size

If any packages had changed sizes they would be listed here.

Changeset



Full Asset Analysis (Modern)

Asset Size Report
=================


Library: EmberData
┌────────────┬──────────┐
│  (index)   │  Values  │
├────────────┼──────────┤
│   bytes    │ '0.00 B' │
│ compressed │ '1.00 B' │
│  packages  │    0     │
│  modules   │    0     │
└────────────┴──────────┘

Modern Builds (No Rollup)

☑️ EmberData has not changed in size

If any packages had changed sizes they would be listed here.

Changeset



Full Asset Analysis (Modern)

Asset Size Report
=================


Library: EmberData
┌────────────┬──────────┐
│  (index)   │  Values  │
├────────────┼──────────┤
│   bytes    │ '0.00 B' │
│ compressed │ '1.00 B' │
│  packages  │    0     │
│  modules   │    0     │
└────────────┴──────────┘

@runspired runspired merged commit 94d233d into master Jul 15, 2022
@delete-merged-branch delete-merged-branch bot deleted the runspired/cleanup-deps branch July 15, 2022 19:57
},
"engines": {
"node": "12.* || >= 14.*"
"node": "12.* || >= 14.*",
"yarn": "1.22.19"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this change is causing issues for people installing ember-data with older yarn versions:

ember-data@4.6.0: The engine "yarn" is incompatible with this module. Expected version "1.22.19". Got "1.22.18"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah. I forgot that is going to get applied externally. We should remove it then. Annoying cause it's nice for folks that don't use Volta.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎯 canary PR is targeting canary (default) 🏷️ chore This PR primarily refactors code or updates dependencies 🏷️ test This PR primarily adds tests for a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants