Skip to content

🔄 Synced file(s) with jhudsl/AnVIL_Template#27

Merged
avahoffman merged 4 commits intomainfrom
repo-sync/AnVIL_Template/default
Feb 18, 2025
Merged

🔄 Synced file(s) with jhudsl/AnVIL_Template#27
avahoffman merged 4 commits intomainfrom
repo-sync/AnVIL_Template/default

Conversation

@cansavvy
Copy link
Copy Markdown
Contributor

@cansavvy cansavvy commented Jan 15, 2025

Synced local file(s) with jhudsl/AnVIL_Template.

Changed files
  • Synced local _output.yml with remote _output.yml

This PR was created automatically by the repo-file-sync-action workflow run #13012693125

@cansavvy cansavvy added the sync label Jan 15, 2025
@cansavvy cansavvy force-pushed the repo-sync/AnVIL_Template/default branch from 38fb9d1 to a6987ce Compare January 28, 2025 14:33
@kweav
Copy link
Copy Markdown
Contributor

kweav commented Feb 14, 2025

@avahoffman the changes look like they change important information. Should this PR be closed without merging changes?

@kweav kweav requested a review from avahoffman February 14, 2025 20:41
@KatherineCox
Copy link
Copy Markdown

@kweav there's nothing important coming from AnVIL_Template in this PR, so it's fine to just close it

@avahoffman is this repo intentionally not syncing workflows? Right now it's set up to only receive changes to _output.yml, but not the main set of sync files (workflows, assets, etc.)

@avahoffman
Copy link
Copy Markdown
Contributor

This repo has some different behavior enabling the rendering of the Collection as part of a workflow. This gets removed with each sync which is a bit annoying. However, it's probably a good idea to have this repo to receive updates, since there are some changes to the checkout action we'll want to incorporate.

the feedback link is created automatically for each repo so merging this PR won't break anything. Although it will be kind of silly to change it and then change it back.

@avahoffman avahoffman merged commit 8270287 into main Feb 18, 2025
2 checks passed
@avahoffman avahoffman deleted the repo-sync/AnVIL_Template/default branch February 18, 2025 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants