Skip to content

Fix client release error#139

Merged
andresfchamorro merged 1 commit into
mainfrom
fix/ci-widgets-dependencies
Jul 2, 2025
Merged

Fix client release error#139
andresfchamorro merged 1 commit into
mainfrom
fix/ci-widgets-dependencies

Conversation

@Gabe-Levin
Copy link
Copy Markdown
Collaborator

@Gabe-Levin Gabe-Levin commented Jul 2, 2025

What I changed:

To fix the dependency error in the client-release test workflow for the widgets, I moved the install command to its own line and specified installation in editable mode.

poetry run pip install -e .[widgets]

Additionally, I added a workflow_dispatch in order to test the workflow without triggering an actual release. All tests passed after running the following:

gh workflow run "Client Release" --ref fix/ci-widgets-dependencies

The test run can be seen in the actions tab on github here: https://github.com/worldbank/DECAT_Space2Stats/actions/runs/16023331907

Clicking on this action, you can see that the widget dependencies (ie. ipywidgets etc) were successfully installed, and all the tests passed.

@Gabe-Levin Gabe-Levin temporarily deployed to Space2Stats API Dev July 2, 2025 10:58 — with GitHub Actions Inactive
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 2, 2025

PR Deployment Details:
🚀 PR deployed to https://fnascf2tl7.execute-api.us-east-1.amazonaws.com/

@Gabe-Levin Gabe-Levin changed the title Fix release error Fix client release error Jul 2, 2025
@andresfchamorro andresfchamorro self-requested a review July 2, 2025 12:44
@andresfchamorro andresfchamorro merged commit 57b130c into main Jul 2, 2025
9 checks passed
@andresfchamorro andresfchamorro temporarily deployed to Space2Stats API Dev July 2, 2025 12:46 — with GitHub Actions Inactive
@andresfchamorro andresfchamorro deleted the fix/ci-widgets-dependencies branch July 2, 2025 12:46
@andresfchamorro andresfchamorro temporarily deployed to Space2Stats API Dev July 2, 2025 12:48 — with GitHub Actions Inactive
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