Skip to content

Refactor to use pip installation function from neon-utils#761

Merged
NeonDaniel merged 5 commits into
devfrom
REF_NeonUtilsPipInstallation
Sep 11, 2025
Merged

Refactor to use pip installation function from neon-utils#761
NeonDaniel merged 5 commits into
devfrom
REF_NeonUtilsPipInstallation

Conversation

@NeonDaniel
Copy link
Copy Markdown
Member

@NeonDaniel NeonDaniel commented Sep 3, 2025

Description

Deprecates logic duplicated in neon-utils for managing pip installation with constraints
Includes update to Docker default configuration to prevent logged errors in default container load

Issues

Other Notes

  • Deployed as ghcr.io/neongeckocom/neon_skills-default_skills:alpha Docker image for testing (9/3)
  • This change means that an invalid default_skills configuration value will cause no skills to be installed, rather than just the one skill to not be installed
  • Previously, URLs that weren't git repositories were treated as invalid, but this removes that check which allows for specifying a wheel URL

@NeonDaniel NeonDaniel requested a review from mikejgray September 3, 2025 19:17
@NeonDaniel NeonDaniel force-pushed the REF_NeonUtilsPipInstallation branch from d48ad8e to b5c3e79 Compare September 11, 2025 01:10
@NeonDaniel NeonDaniel marked this pull request as ready for review September 11, 2025 03:03
@NeonDaniel NeonDaniel merged commit 1ccf3fc into dev Sep 11, 2025
26 of 30 checks passed
@NeonDaniel NeonDaniel deleted the REF_NeonUtilsPipInstallation branch September 11, 2025 03:54
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.

1 participant