Skip to content

Fix forge oauth columns migration#5525

Merged
xoxys merged 3 commits into
mainfrom
fix-migration
Sep 19, 2025
Merged

Fix forge oauth columns migration#5525
xoxys merged 3 commits into
mainfrom
fix-migration

Conversation

@xoxys

@xoxys xoxys commented Sep 19, 2025

Copy link
Copy Markdown
Member

Regression introduced in #5517

@xoxys xoxys requested a review from a team September 19, 2025 18:11
@xoxys xoxys added server regression fix a bug that was not released yet labels Sep 19, 2025
@codecov

codecov Bot commented Sep 19, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 66.66667% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 26.41%. Comparing base (30d33e1) to head (0456e08).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...store/datastore/migration/027_fix_forge_columns.go 66.66% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5525      +/-   ##
==========================================
- Coverage   26.41%   26.41%   -0.01%     
==========================================
  Files         405      405              
  Lines       28869    28878       +9     
==========================================
+ Hits         7627     7628       +1     
- Misses      20544    20551       +7     
- Partials      698      699       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@xoxys xoxys merged commit 7800d13 into main Sep 19, 2025
9 checks passed
@xoxys xoxys deleted the fix-migration branch September 19, 2025 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

regression fix a bug that was not released yet server

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants