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

Change nextjs detection in vc build #7020

Merged
merged 2 commits into from
Nov 23, 2021
Merged

Change nextjs detection in vc build #7020

merged 2 commits into from
Nov 23, 2021

Conversation

jaredpalmer
Copy link
Contributor

@jaredpalmer jaredpalmer commented Nov 16, 2021

Instead of trusting framework settings in .vercel/project.json, use the presence of .next directory after the build command is run

@codecov
Copy link

codecov bot commented Nov 16, 2021

Codecov Report

Merging #7020 (adf9aa0) into main (7d7f3df) will not change coverage.
The diff coverage is n/a.

❗ Current head adf9aa0 differs from pull request most recent head 986b460. Consider uploading reports for the commit 986b460 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##             main    #7020   +/-   ##
=======================================
  Coverage   51.49%   51.49%           
=======================================
  Files         128      128           
  Lines        4979     4979           
  Branches     1172     1172           
=======================================
  Hits         2564     2564           
  Misses       2407     2407           
  Partials        8        8           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7d7f3df...986b460. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants