Skip to content
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

[Maintenance] Update CIrcleCI redirector circleci.yml to fix artifact path #278

Merged
merged 2 commits into from
Nov 21, 2023

Conversation

psobolewskiPhD
Copy link
Member

References and relevant issues

The CircleCI redirector is not re-directing properly even when build succeeds, see:
#273 (comment)

Description

In #266 the artifact path got changed. This fixes the redirector action to use the new path.

@psobolewskiPhD psobolewskiPhD added this to the 0.5.0 milestone Nov 20, 2023
@github-actions github-actions bot added the task label Nov 20, 2023
@psobolewskiPhD psobolewskiPhD added maintenance CI, dependencies, and other maintenance and removed task labels Nov 20, 2023
@psobolewskiPhD psobolewskiPhD changed the title Update circleci.yml to fix artifact path [Maintenance] Update circleci.yml to fix artifact path Nov 20, 2023
@psobolewskiPhD psobolewskiPhD changed the title [Maintenance] Update circleci.yml to fix artifact path [Maintenance] Update CIrcleCI redirector circleci.yml to fix artifact path Nov 20, 2023
@psobolewskiPhD
Copy link
Member Author

Hopefully this fixes it and if so I'll make PR to napari/napari where the same issue occurs.

Copy link
Member

@melissawm melissawm left a comment

Choose a reason for hiding this comment

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

I see - thanks for the fix, let's see if this helps 🙏

@github-actions github-actions bot added the task label Nov 21, 2023
@psobolewskiPhD
Copy link
Member Author

psobolewskiPhD commented Nov 21, 2023

fixed it -- the 0 is needed, see the links to the artifacts:
https://output.circle-artifacts.com/output/job/e3ff28b4-0da3-45d3-83e4-b80cf90e43ec/artifacts/0/docs/docs/_build/index.html

But I think this needs to be merged to see it run on CI, so I will optimistically mark ready-to-merge and we'll see and fix as needed.

@psobolewskiPhD
Copy link
Member Author

Looks like everything works fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance CI, dependencies, and other maintenance task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants