Skip to content

Conversation

@carriewright11
Copy link
Contributor

@carriewright11 carriewright11 commented May 12, 2025

I noticed that the https://github.com/fhdsl/DaSL_Collection/actions/workflows/build-collection.yml has never run. The plan I think was to be triggered by the other workflows, but I may be misinterpreting.

I am adding a dispatch option in case we want to manually trigger (this is helpful for testing changes in the collection or the code to generate the collection). I wonder if a schedule makes sense for this workflow too (like the render wokflow).

@github-actions
Copy link

github-actions bot commented May 12, 2025

No spelling errors! 🎉
Comment updated at 2025-05-12-20:01:34 with changes from b92adf5

@github-actions
Copy link

github-actions bot commented May 12, 2025

No broken url errors! 🎉
Comment updated at 2025-05-12-20:01:33 with changes from b92adf5

@carriewright11
Copy link
Contributor Author

@avahoffman wanted to make you aware of what I was doing. What do you think about having the build-collection.yml get triggered on a schedule (like the render-site.yml)?

@github-actions
Copy link

Re-rendered previews from the latest commit:

Updated at 2025-05-12 with changes from b92adf5

@carriewright11 carriewright11 merged commit fa844b0 into main May 12, 2025
12 checks passed
@carriewright11 carriewright11 deleted the carriewright11-patch-1 branch May 12, 2025 20:04
@avahoffman
Copy link
Contributor

THanks! totally makes sense.

@carriewright11
Copy link
Contributor Author

@avahoffman i see now that it is triggered by the render-site.yml file

@carriewright11
Copy link
Contributor Author

so actually maybe we don't need this anymore

@avahoffman
Copy link
Contributor

Ah yes, you're right, maybe we revert then?

@carriewright11
Copy link
Contributor Author

yup - im on it :)

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.

3 participants