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

Package name: cihai[cli] -> cihai-cli #279

Merged
merged 2 commits into from Aug 20, 2022
Merged

Package name: cihai[cli] -> cihai-cli #279

merged 2 commits into from Aug 20, 2022

Conversation

tony
Copy link
Member

@tony tony commented Aug 20, 2022

cihai/cihai#326, #279

The CLI version of cihai installed through cihai-cli again

Old (cihai 0.9 to 0.14, cihai-cli 0.5 to 0.10):

$ pip install cihai[cli]

After (cihai 0.15+, cihai-cli 0.11+):

$ pip install cihai-cli

This made deploying cihai + cihai-cli and pinning packages extremely laborious.

We can reinvestigate this model in the future.

Having to use an extra package made it much harder to pin and deploy
packages. We may look into this again in the future.
@codecov
Copy link

codecov bot commented Aug 20, 2022

Codecov Report

Merging #279 (7948273) into master (5c5b35b) will not change coverage.
The diff coverage is n/a.

❗ Current head 7948273 differs from pull request most recent head 6598b09. Consider uploading reports for the commit 6598b09 to get more accurate results

@@           Coverage Diff           @@
##           master     #279   +/-   ##
=======================================
  Coverage   75.44%   75.44%           
=======================================
  Files           5        5           
  Lines         224      224           
=======================================
  Hits          169      169           
  Misses         55       55           

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

@tony tony force-pushed the remove-extras-package branch 2 times, most recently from 7948273 to 6598b09 Compare August 20, 2022 16:00
@tony tony merged commit f4d7076 into master Aug 20, 2022
@tony tony deleted the remove-extras-package branch August 20, 2022 16:01
tony added a commit to cihai/cihai that referenced this pull request Aug 20, 2022
**#326, cihai/cihai-cli#279

The CLI version of `cihai` installed through `cihai-cli` again

Old (cihai 0.9 to 0.14, cihai-cli 0.5 to 0.10):

```console
$ pip install cihai[cli]
```

After (cihai 0.15+, cihai-cli 0.11+):

```console
$ pip install cihai-cli
```

This made deploying cihai + cihai-cli and pinning packages extremely laborious.

We can reinvestigate this model in the future.
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.

None yet

1 participant