-
Notifications
You must be signed in to change notification settings - Fork 91
test: adjust tests for next@16 #3169
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
Conversation
4561a34
to
ee4f245
Compare
📊 Package size report No changes
Unchanged files
🤖 This report was automatically generated by pkg-size-action |
const shouldTestAllVersions = labels.some(label => label.name === 'autorelease: pending' || label.name === 'test all versions'); | ||
if (shouldTestAllVersions) { | ||
return 'all' | ||
} | ||
return labels.some(label => label.name === 'test latest and canary') ? 'latest-and-canary' : 'latest' | ||
- name: Set Next.js versions to test | ||
id: set-matrix | ||
# If this is the nightly build or a release PR then run the full matrix of versions | ||
run: | | ||
if [ "${{ github.event_name }}" == "workflow_dispatch" ]; then | ||
echo "matrix=${{ github.event.inputs.versions }}" >> $GITHUB_OUTPUT | ||
elif [ "${{ github.event_name }}" = "schedule" ] || [ "${{ steps.check-labels.outputs.result }}" = "true" ]; then | ||
elif [ "${{ github.event_name }}" = "schedule" ] || [ "${{ steps.check-labels.outputs.result }}" = "all" ]; then | ||
echo "matrix=[\"latest\", \"canary\", \"14.2.15\", \"13.5.1\"]" >> $GITHUB_OUTPUT | ||
elif [ "${{ steps.check-labels.outputs.result }}" = "latest-and-canary" ]; then | ||
echo "matrix=[\"latest\", \"canary\"]" >> $GITHUB_OUTPUT |
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.
adding way to run just latest
and canary
variants without v13 and v14 to save CI time while working on adjustments for Next@16
fac5db3
to
f4ed061
Compare
f4ed061
to
18f7ee7
Compare
// (14.2.10 and canary.147) | ||
const shouldHavePrivateDirective = nextVersionSatisfies( | ||
'<14.2.4 || >=14.2.10 <15.0.0-canary.24 || ^15.0.0-canary.147', | ||
'<14.2.4 || >=14.2.10 <15.0.0-canary.24 || >=15.0.0-canary.147', |
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.
updated to have v16 fall into version range
experimental: pprConfigHardDeprecated | ||
? { | ||
cacheComponents: true, | ||
} | ||
: { | ||
ppr: true, | ||
}, |
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.
experimental.ppr
hard fails now forcing to use experimental.cacheComponents
This does introduce also introduce some changes to some behaviors and ppr test was adjusted for it as well
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.
LGTM
Description
Details either in code comments or inline comments