Skip to content

Limit contest dump scope - #1730

Open
pxsit wants to merge 3 commits into
cms-dev:mainfrom
pxsit:bugfix2
Open

pxsit wants to merge 3 commits into
cms-dev:mainfrom
pxsit:bugfix2

Conversation

@pxsit

@pxsit pxsit commented Oct 17, 2026

Copy link
Copy Markdown
Contributor

Prevent exporting shared users and teams from pulling in participations from unrelated contests. The importer now traverses only objects rooted in selected contests, filtering unrelated references from older dumps.

Prevent exporting shared users and teams from pulling in participations from unrelated contests. The importer now traverses only objects rooted in selected contests, filtering unrelated references from older dumps.
Copilot AI balanced review requested due to automatic review settings October 17, 2026 14:57

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.

Reuse destination administrators matched by username during dump import so their local credentials and permissions are preserved.
@codecov

codecov Bot commented Oct 17, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 85.29412% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 56.75%. Comparing base (3277ca5) to head (0ef4405).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
cmscontrib/DumpImporter.py 84.37% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1730      +/-   ##
==========================================
+ Coverage   56.72%   56.75%   +0.03%     
==========================================
  Files         344      344              
  Lines       28314    28346      +32     
==========================================
+ Hits        16060    16088      +28     
- Misses      12254    12258       +4     
Flag Coverage Δ
functionaltests 0.00% <0.00%> (ø)
unittests 56.75% <85.29%> (+0.03%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants