Skip to content

Conversation

@hrideshmg
Copy link
Contributor

Please prefix your pull request with one of the following: [FEATURE] [FIX] [IMPROVEMENT].

In raising this pull request, I confirm the following (please check boxes):

  • I have read and understood the contributors guide.
  • I have checked that another pull request for this purpose does not exist.
  • I have considered, and confirmed that this submission will be valuable to others.
  • I accept that this submission may not be used, and the pull request closed at the will of the maintainer.
  • I give this submission freely, and claim no ownership to its content.

My familiarity with the project is as follows (check one):

  • I have never used the project.
  • I have used the project briefly.
  • I have used the project extensively, but have not contributed previously.
  • I am an active contributor to the project.

While debugging the windows tests i noticed that a fair amount of time was spent copying the files from the mounted TestResultsRemote directory to the actually utilized TestResults directory.

Replacing the mount + file copy operation with an rclone copy step reduced the VM startup time by around 10 mins since the latter is optimized for bulk file transfers.

The windows defender exclusion is necessary because windows was flagging rclone as a trojan while copying files for some reason, see here

@codecov
Copy link

codecov bot commented May 31, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.82%. Comparing base (fd80f8f) to head (1693d5a).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #929      +/-   ##
==========================================
- Coverage   86.85%   86.82%   -0.03%     
==========================================
  Files          35       35              
  Lines        3705     3705              
  Branches      759      448     -311     
==========================================
- Hits         3218     3217       -1     
- Misses        351      352       +1     
  Partials      136      136              
Flag Coverage Δ
unittests 86.82% <ø> (-0.03%) ⬇️

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

☔ 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.

@sonarqubecloud
Copy link

@canihavesomecoffee canihavesomecoffee merged commit 6186a31 into CCExtractor:master May 31, 2025
4 of 12 checks passed
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.

2 participants