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

Add template-files composite action #127

Merged
merged 27 commits into from
May 8, 2024
Merged

Add template-files composite action #127

merged 27 commits into from
May 8, 2024

Conversation

kenodegard
Copy link
Contributor

Description

We've been using github.com/BetaHuhn/repo-file-sync-action to synchronize files from conda/infrastructure for a while now and have found that it doesn't perform in the way we need it to (bulk syncing with templating), the upstream author is not particularly responsive anymore, and since the upstream action is written in NodeJS/Typescript its not particularly simple to fork and fix.

Thus we implement a small composite action that implements a pull process and uses Jinja as the templating engine. Since this is a decentralized pull process this also gives more control to individual repos instead of having a central sync conductor.

@kenodegard kenodegard self-assigned this Sep 25, 2023
@conda-bot conda-bot added the cla-signed [bot] added once the contributor has signed the CLA label Sep 25, 2023
@jezdez jezdez self-requested a review May 7, 2024 08:46
jezdez
jezdez previously approved these changes May 7, 2024
Copy link
Member

@jezdez jezdez left a comment

Choose a reason for hiding this comment

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

Some nits on naming etc but I think that's a great start!

template-files/action.py Show resolved Hide resolved
template-files/action.yml Show resolved Hide resolved
template-files/action.yml Outdated Show resolved Hide resolved
template-files/action.yml Outdated Show resolved Hide resolved
template-files/action.py Outdated Show resolved Hide resolved
template-files/action.py Outdated Show resolved Hide resolved
template-files/action.py Outdated Show resolved Hide resolved
template-files/action.py Show resolved Hide resolved
template-files/action.yml Outdated Show resolved Hide resolved
template-files/action.yml Outdated Show resolved Hide resolved
@kenodegard kenodegard force-pushed the template-files branch 10 times, most recently from 4ebc5d2 to 4653fa7 Compare May 7, 2024 19:17
@kenodegard
Copy link
Contributor Author

I have verified that the workflow still works in https://github.com/conda-sandbox:

See a scheduled run (where new updates are fetched from upstream): conda-sandbox/downstream#19
And a manually triggered run (where we enable nested templating features): conda-sandbox/downstream#20

@jezdez jezdez merged commit 976289d into main May 8, 2024
3 checks passed
@jezdez jezdez deleted the template-files branch May 8, 2024 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed [bot] added once the contributor has signed the CLA
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants