Skip to content
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

Add support for synchronous and asynchronous modes in package cache #1853

Conversation

Pixel-Minions
Copy link
Contributor

  • Introduced a new command-line argument --pkg-cache-mode to specify caching mode.
  • Allows users to choose between 'sync' and 'async' modes, overriding the default configuration.
  • Updated add_variant function to handle the new caching mode option.
  • Ensures backward compatibility by maintaining default behavior when no mode is specified.

@Pixel-Minions Pixel-Minions requested a review from a team as a code owner October 14, 2024 18:09
Copy link

codecov bot commented Oct 14, 2024

Codecov Report

Attention: Patch coverage is 12.50000% with 7 lines in your changes missing coverage. Please review.

Project coverage is 59.28%. Comparing base (491497f) to head (8fcefd3).
Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
src/rez/cli/pkg-cache.py 12.50% 7 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1853      +/-   ##
==========================================
- Coverage   59.30%   59.28%   -0.02%     
==========================================
  Files         126      126              
  Lines       17210    17217       +7     
  Branches     3015     3017       +2     
==========================================
+ Hits        10206    10207       +1     
- Misses       6319     6325       +6     
  Partials      685      685              

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

@Pixel-Minions Pixel-Minions force-pushed the Add-support-for-synchronous-mode-in-rez-pkg-cache branch from 8fc7b01 to c21b725 Compare October 14, 2024 18:13
- Introduced a new command-line argument `--pkg-cache-mode` to specify caching mode.
- Allows users to choose between 'sync' and 'async' modes, overriding the default configuration.
- Updated `add_variant` function to handle the new caching mode option.
- Ensures backward compatibility by maintaining default behavior when no mode is specified.
Signed-off-by: Jose Enriquez <alenrqz21@gmail.com>
@Pixel-Minions Pixel-Minions force-pushed the Add-support-for-synchronous-mode-in-rez-pkg-cache branch from c21b725 to b49ef51 Compare October 14, 2024 18:15
Signed-off-by: Jose Enriquez <alenrqz21@gmail.com>

Co-authored-by: Jean-Christophe Morin <38703886+JeanChristopheMorinPerso@users.noreply.github.com>
Signed-off-by: Jose Enriquez <1145623+Pixel-Minions@users.noreply.github.com>
Signed-off-by: Jose Enriquez <alenrqz21@gmail.com>

Co-authored-by: Jean-Christophe Morin <38703886+JeanChristopheMorinPerso@users.noreply.github.com>
Signed-off-by: Jose Enriquez <1145623+Pixel-Minions@users.noreply.github.com>
Signed-off-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
Signed-off-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
Copy link
Member

@JeanChristopheMorinPerso JeanChristopheMorinPerso left a comment

Choose a reason for hiding this comment

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

Thank you @Pixel-Minions!

@JeanChristopheMorinPerso JeanChristopheMorinPerso merged commit effa516 into AcademySoftwareFoundation:main Oct 19, 2024
45 of 46 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