-
Notifications
You must be signed in to change notification settings - Fork 5
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
Hotfix - set limit to 100 again #2117
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2117 +/- ##
=======================================
Coverage 83.29% 83.29%
=======================================
Files 100 100
Lines 3580 3580
=======================================
Hits 2982 2982
Misses 598 598
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Indøk Web Run #3353
Run Properties:
|
Project |
Indøk Web
|
Branch Review |
Pages-order-info
|
Run status |
Failed #3353
|
Run duration | 01m 25s |
Commit |
b13cb442ee ℹ️: Merge 2801e195a683092ab79cb1829f4b1ae44764378a into 07dd5c4eaab8678d7af4ea8fd55e...
|
Committer | Erik Aas |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
1
|
Flaky |
0
|
Pending |
0
|
Skipped |
0
|
Passing |
1
|
View all changes introduced in this branch ↗︎ |
Tests for review
cypress/e2e/landing/feide.cy.ts • 1 failed test • E2E
Test | Artifacts | |
---|---|---|
SSO login > should prompt registration when not registered |
Test Replay
Screenshots
|
In the future if you hotfix something the PR should say: Also if a commit fixes something in production it should not have new features. It makes it harder to merge quickly as the reviewer needs to look through the whole request |
Quality Gate passedIssues Measures |
Changes