Skip to content

feat: modpacks#125

Draft
Vilsol wants to merge 82 commits intostagingfrom
feat/modpacks
Draft

feat: modpacks#125
Vilsol wants to merge 82 commits intostagingfrom
feat/modpacks

Conversation

@Vilsol
Copy link
Copy Markdown
Member

@Vilsol Vilsol commented Jun 24, 2025


Rob's list of assorted Rose To-Do's before this can be merged:

  • Make real name for 20260321003544 migration
  • Review all added code comments to ensure they are still relevant. If they suggest future work/changes, ideally turn them into github issues with links to that line in the code so we can discuss them later

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 24, 2025

Codecov Report

❌ Patch coverage is 57.30337% with 38 lines in your changes missing coverage. Please review.
✅ Project coverage is 43.67%. Comparing base (7b96fac) to head (617294b).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
dataloader/loaders.go 0.00% 14 Missing ⚠️
db/user.go 0.00% 11 Missing ⚠️
db/modpack.go 76.92% 5 Missing and 4 partials ⚠️
conversion/ent_to_graphql.go 82.60% 2 Missing and 2 partials ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           staging     #125      +/-   ##
===========================================
+ Coverage    41.73%   43.67%   +1.93%     
===========================================
  Files          128      113      -15     
  Lines         6146     6592     +446     
===========================================
+ Hits          2565     2879     +314     
- Misses        3191     3257      +66     
- Partials       390      456      +66     

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

@Vilsol Vilsol requested review from budak7273 and mircearoata June 24, 2025 23:41
@budak7273 budak7273 marked this pull request as draft April 4, 2026 03:13
rhit-zhangl8 and others added 25 commits April 27, 2026 18:31
…ts on release creation, added query for calculating lockfile & targets. tests may fail due to schema change
…tors to check their changes against the most recent staging dump
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants