-
Notifications
You must be signed in to change notification settings - Fork 103
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
Enable CCD update from installed package #716
Conversation
This sound indeed like a good idea, I have not thought about, yet. From my perspective the most important point is, that it cleans up the install-setup-install dance required now. I will give it a few days to think about possible caveats, we might have missed. But otherwise I think this should definitely go into the upcoming release. |
If you have no objections I could also finalize this PR, including the small adjustments required in the CI. |
that would be teriffic @padix-key ! yes please make the adjustments as needed FYI I went ahead and made the changes you suggested but the install readme could probably be amended further for clarity |
@padix-key btw, any idea when you might want to cut a new release? |
I plan the next release before the end of the year, maybe already this weekend 😃 . |
36310e5
to
7e92135
Compare
CodSpeed Performance ReportMerging #716 will not alter performanceComparing Summary
|
584a079
to
41f1037
Compare
I added the remaining necessary changes. Thanks again! |
currently the setup_ccd.py script has to be run from a clone of the repo. this creates a command-line utility to run the ccd update directly from the pip installed package itself. i think this simplifies dependency management for downstream projects quite a bit