Skip to content
This repository has been archived by the owner on Sep 13, 2023. It is now read-only.

--quiet option #448

Merged
merged 3 commits into from
Oct 27, 2022
Merged

--quiet option #448

merged 3 commits into from
Oct 27, 2022

Conversation

mike0sv
Copy link
Contributor

@mike0sv mike0sv commented Oct 25, 2022

Add --quiet option to suppress output in CLI

@mike0sv mike0sv requested a review from a team as a code owner October 25, 2022 12:13
@mike0sv mike0sv self-assigned this Oct 25, 2022
@mike0sv mike0sv temporarily deployed to internal October 25, 2022 12:13 Inactive
@mike0sv mike0sv temporarily deployed to internal October 25, 2022 12:13 Inactive
@aguschin aguschin mentioned this pull request Oct 27, 2022
18 tasks
@aguschin
Copy link
Contributor

@madhur-tandon, can you please fix the miniconda thing? Otherwise we can't check if this works.

@mike0sv mike0sv temporarily deployed to internal October 27, 2022 10:55 Inactive
@aguschin aguschin temporarily deployed to internal October 27, 2022 10:55 Inactive
@codecov
Copy link

codecov bot commented Oct 27, 2022

Codecov Report

Base: 82.98% // Head: 86.87% // Increases project coverage by +3.89% 🎉

Coverage data is based on head (46157b0) compared to base (de9c700).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@                Coverage Diff                @@
##           release/0.3.0     #448      +/-   ##
=================================================
+ Coverage          82.98%   86.87%   +3.89%     
=================================================
  Files                 96       96              
  Lines               7882     7880       -2     
=================================================
+ Hits                6541     6846     +305     
+ Misses              1341     1034     -307     
Impacted Files Coverage Δ
mlem/cli/main.py 84.14% <100.00%> (-0.13%) ⬇️
mlem/core/requirements.py 95.45% <0.00%> (+0.34%) ⬆️
mlem/core/objects.py 85.54% <0.00%> (+2.58%) ⬆️
mlem/api/utils.py 78.18% <0.00%> (+5.45%) ⬆️
mlem/api/commands.py 83.33% <0.00%> (+6.06%) ⬆️
mlem/ui.py 94.66% <0.00%> (+8.00%) ⬆️
mlem/contrib/kubernetes/service.py 51.72% <0.00%> (+10.34%) ⬆️
mlem/contrib/kubernetes/build.py 100.00% <0.00%> (+30.00%) ⬆️
mlem/contrib/kubernetes/base.py 83.87% <0.00%> (+39.78%) ⬆️
mlem/contrib/docker/base.py 84.36% <0.00%> (+41.45%) ⬆️
... and 4 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@madhur-tandon
Copy link
Contributor

@madhur-tandon, can you please fix the miniconda thing? Otherwise we can't check if this works.

Done in #452

@aguschin aguschin temporarily deployed to internal October 27, 2022 11:50 Inactive
@aguschin
Copy link
Contributor

mmm. Failing too often, again?

@aguschin
Copy link
Contributor

Merging this since it doesn't have anything to do with Heroku.

@aguschin aguschin merged commit 601be82 into release/0.3.0 Oct 27, 2022
@aguschin aguschin deleted the feature/quiet branch October 27, 2022 12:06
aguschin added a commit that referenced this pull request Oct 27, 2022
This is a list of activities that should be done before release announce

## First, things before release

### Breaking changes - merge in this PR
- [x] New deployment state management
- [x] New CLI interface: breaking changes in `serve`, `apply-remote`,
`build` and `declare`

### Other changes - merge in this PR
- [x] SageMaker deployments
- [x] K8s deployments
- [x] `--quiet` #448 @mike0sv 

### Docs
- [x] Finish semi-generated API reference PR
iterative/mlem.ai#172
- [x] Update Quick Start in this repo
- [x] iterative/mlem.ai#197
- [ ] iterative/mlem.ai#188

### Blog post
- [ ] blog post re release @aguschin 

### Example-repo - PR in `example-repos-dev`
- [x] update `example-mlem-get-started`

### Update the website code animations
- [x] https://mlem.ai

## Second, the release itself

### MLEM Release
- [ ] merge this branch
- [ ] release it on PyPi
- [ ] merge docs PR
- [ ] merge PR in `example-repos-dev` and update
`iterative/example-mlem-get-started`

## Third, things after release

- [ ] remove `xfail` mark from
`tests/cli/test_mail.py::test_commands_docs_links`
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants