Skip to content

fix: shrink docker image by excluding dev deps and pruning sktime's s… - #2438

Open
TeaDrinkingProgrammer wants to merge 1 commit into
FlexMeasures:mainfrom
TeaDrinkingProgrammer:fix/docker-image-size
Open

fix: shrink docker image by excluding dev deps and pruning sktime's s…#2438
TeaDrinkingProgrammer wants to merge 1 commit into
FlexMeasures:mainfrom
TeaDrinkingProgrammer:fix/docker-image-size

Conversation

@TeaDrinkingProgrammer

Copy link
Copy Markdown
Contributor

…tray docs/examples

Description

This PR will shrink the Docker image by a few tens of MBs by excluding dev dependencies and pruning sktime examples (see sktime/sktime#10891)

  • Added changelog item in documentation/changelog.rst

Look & Feel

...

How to test

...

Further Improvements

...

Related Items

...


Sign-off

  • I agree to contribute to the project under Apache 2 License.
  • To the best of my knowledge, the proposed patch is not based on code under GPL or other license that is incompatible with FlexMeasures

…tray docs/examples

Signed-off-by: Stijn van Houwelingen <teadrinkingprogrammer@proton.me>
@read-the-docs-community

Copy link
Copy Markdown

Documentation build overview

📚 flexmeasures | 🛠️ Build #34219663 | 📁 Comparing 0b697c7 against latest (3272084)

  🔍 Preview build  

2 files changed
± changelog.html
± api/v3_0.html

@Flix6x Flix6x left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice. I'm looking into the other suggestions, but I think this can go in a right away.


Infrastructure / Support
-------------------------
* Shrink the Docker image by excluding dev-only dependencies and pruning stray ``docs``/``examples`` payloads bundled by ``sktime``/``scikit-base`` (issue: https://github.com/sktime/sktime/issues/10891) [see `PR #XXXX <https://www.github.com/FlexMeasures/flexmeasures/pull/XXXX>`_]

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please fill in the PR number (twice).

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