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

Extra parameters for quilt3 push #3406

Draft
wants to merge 19 commits into
base: master
Choose a base branch
from
Draft

Extra parameters for quilt3 push #3406

wants to merge 19 commits into from

Conversation

sir-sigurd
Copy link
Member

@sir-sigurd sir-sigurd commented Apr 10, 2023

Description

TODO

  • it's not clear how these flags should affect --meta
  • Unit tests
  • Documentation
  • Changelog entry (skip if change is not significant to end users, e.g. docs only)

@codecov
Copy link

codecov bot commented Apr 10, 2023

Codecov Report

Merging #3406 (018b113) into master (6bed089) will decrease coverage by 26.21%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #3406       +/-   ##
==========================================
- Coverage   35.78%   9.57%   -26.21%     
==========================================
  Files         680     601       -79     
  Lines       29691   19908     -9783     
  Branches     4377    4377               
==========================================
- Hits        10626    1907     -8719     
+ Misses      17902   16838     -1064     
  Partials     1163    1163               
Flag Coverage Δ
api-python ?
catalog 9.57% <ø> (ø)
lambda ?

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

see 79 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@sir-sigurd sir-sigurd requested a review from drernie April 10, 2023 13:07
api/python/quilt3/main.py Outdated Show resolved Hide resolved
api/python/quilt3/main.py Outdated Show resolved Hide resolved
api/python/quilt3/main.py Outdated Show resolved Hide resolved
drernie
drernie previously approved these changes Apr 11, 2023
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.

None yet

2 participants