Skip to content

Refactor skill installation helpers - #968

Merged
jana-selva merged 6 commits into
mainfrom
feature/967-refactor-skill-installation
Oct 14, 2026
Merged

jana-selva merged 6 commits into
mainfrom
feature/967-refactor-skill-installation

Conversation

@jana-selva

Copy link
Copy Markdown
Contributor

Closes #967

Stacked on PR #966 (feature/963-multi-skill-support).

This follow-up incorporates the remaining review context from PR #966 and the api-contract-audit skill review:

Validation:

  • 38 focused skill tests passed.
  • skills:check passed.
  • Project-template skill integration test passed on retry.
  • Ruff and git diff --check passed.

@jana-selva
jana-selva added this pull request to stack #969 September 30, 2026 11:18
@jana-selva jana-selva self-assigned this Sep 30, 2026
@jana-selva
jana-selva deployed to manual-approval September 30, 2026 11:20 — with GitHub Actions Active
@jana-selva
jana-selva deployed to manual-approval September 30, 2026 11:20 — with GitHub Actions Active
@ArBridgeman
ArBridgeman force-pushed the feature/967-refactor-skill-installation branch from 79f1c22 to ded951f Compare October 5, 2026 11:10
@jana-selva
jana-selva force-pushed the feature/967-refactor-skill-installation branch from ded951f to 4464006 Compare October 13, 2026 06:56
@jana-selva
jana-selva deployed to manual-approval October 13, 2026 06:56 — with GitHub Actions Active
@jana-selva
jana-selva deployed to manual-approval October 13, 2026 06:56 — with GitHub Actions Active
Base automatically changed from feature/963-multi-skill-support to main October 13, 2026 08:42
@ArBridgeman
ArBridgeman force-pushed the feature/967-refactor-skill-installation branch from 4464006 to 540436a Compare October 13, 2026 08:42
@ArBridgeman
ArBridgeman deployed to manual-approval October 13, 2026 08:42 — with GitHub Actions Active
@ArBridgeman
ArBridgeman deployed to manual-approval October 13, 2026 08:42 — with GitHub Actions Active
Comment thread exasol/toolbox/util/skills.py Outdated
Comment thread test/unit/util/skill_test.py Outdated
Comment thread test/unit/util/skill_test.py Outdated
Comment thread test/unit/util/skill_test.py Outdated
@jana-selva
jana-selva deployed to manual-approval October 14, 2026 11:29 — with GitHub Actions Active
@jana-selva
jana-selva deployed to manual-approval October 14, 2026 11:29 — with GitHub Actions Active
@sonarqubecloud

Copy link
Copy Markdown

@jana-selva
jana-selva merged commit fd53f47 into main Oct 14, 2026
41 checks passed
@jana-selva
jana-selva deleted the feature/967-refactor-skill-installation branch October 14, 2026 12:12

This branch was successfully deployed

1 active deployment
manual-approval — 6ab79106 Deployed Oct 14, 2026 by jana-selva via Merge Gate / Approve Running Slow Tests? #2870
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.

Refactor install_skill into reusable filesystem helpers

2 participants