-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Move Go Integration tests into main test Github workflow #3429
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
Move Go Integration tests into main test Github workflow #3429
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 10 Ignored Deployments
|
Failing next.js integration test suitesCommit: 580d9d1
Expand output● ReactRefreshLogBox › Node.js builtins
● ReactRefreshLogBox › Module not found
● ReactRefreshLogBox › Module not found (empty import trace)
● ReactRefreshLogBox › Module not found (missing global CSS)
Expand output● legacyBrowsers: true › should apply legacyBrowsers: true
Expand output● 404 Page Support with _app › dev mode › should not show pages/404 GIP error if _app has GIP
Expand output● ReactRefreshLogBox app › Node.js builtins
● ReactRefreshLogBox app › Module not found
● ReactRefreshLogBox app › Module not found empty import trace
● ReactRefreshLogBox app › Module not found missing global CSS
Expand output● app-dir alias handling › should handle typescript paths alias correctly
● app-dir alias handling › should resolve css imports from outside with src folder presented
Expand output● Default 404 Page with custom _error › dev mode › should respond to 404 correctly
● Default 404 Page with custom _error › dev mode › should render error correctly
● Default 404 Page with custom _error › dev mode › should render index page normal
Expand output● ReactRefreshLogBox › Node.js builtins
● ReactRefreshLogBox › Module not found
● ReactRefreshLogBox › Module not found (empty import trace)
● ReactRefreshLogBox › Module not found (missing global CSS)
Expand output● legacyBrowsers: true › should apply legacyBrowsers: true
Expand output● 404 Page Support with _app › dev mode › should not show pages/404 GIP error if _app has GIP
Expand output● ReactRefreshLogBox app › Node.js builtins
● ReactRefreshLogBox app › Module not found
● ReactRefreshLogBox app › Module not found empty import trace
● ReactRefreshLogBox app › Module not found missing global CSS
Expand output● app-dir alias handling › should handle typescript paths alias correctly
● app-dir alias handling › should resolve css imports from outside with src folder presented
Expand output● Default 404 Page with custom _error › dev mode › should respond to 404 correctly
● Default 404 Page with custom _error › dev mode › should render error correctly
● Default 404 Page with custom _error › dev mode › should render index page normal
|
Benchmark for f41a42fClick to view benchmark
|
🟢 CI successful 🟢Thanks |
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.
Fine by me
Going to merge this. The Rust benchmarking workflows run on this and they take a long time to run. By the time they finish, the PR is out of date already, so it's going to be very hard to get into a state where auto-merge works. |
Benchmark for 580d9d1Click to view benchmark
|
Introduced when moving go integration tests into our main test workflow: #3429
e2e tests are still separate, there's a separate ticket for those.