Skip to content

[feat] PR with artifact preview#8

Open
sosiristseng wants to merge 12 commits into
JuliaPluto:mainfrom
sosiristseng:PR-preview
Open

[feat] PR with artifact preview#8
sosiristseng wants to merge 12 commits into
JuliaPluto:mainfrom
sosiristseng:PR-preview

Conversation

@sosiristseng

Copy link
Copy Markdown

close #6

@sosiristseng

sosiristseng commented Dec 16, 2021

Copy link
Copy Markdown
Author

However, I kept getting errors on "Comment on PR about the artifact link" github action:

https://github.com/sosiristseng/static-export-template/runs/4544460466?check_suite_focus=true

Edit: the latest commit fixed that

https://github.com/sosiristseng/static-export-template/actions/runs/1586404343

@greimel greimel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This would be nice to have!

Could someone re-trigger CI to verify that the artifacts look good?

name: Add artifact links to pull request and related issues
on:
workflow_run:
workflows: ["Export Pluto notebooks"]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
workflows: ["Export Pluto notebooks"]
workflows: [Export Pluto notebooks]

In the examples you linked to, there are no quotation marks

@greimel

greimel commented Jul 29, 2022

Copy link
Copy Markdown
Contributor

I used your code in greimel/pluto-course-template#15 - thanks!

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.

Notebook preview for PRs

2 participants