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

optionally put assets in separate dir #130

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mstandley-tempus
Copy link

After fetching a release with several artifact files, I want to use a shell script to iterate over those files in a task. Ideally, the shell code would look something like this:

for i in *; do
  do_something_to_an_artifact "$i"
done

The problem is that these artifacts coexist in the same directory with metadata files tag, version, body, timestamp, commit_sha, and url. The * glob will include those metadata files, but I only want it to include the artifacts downloaded from the GitHub release.

So, I propose adding a new boolean source parameter called asset_dir. When enabled, the resource will put the artifacts in a new subdirectory called assets.

I tested my changes with asset_dir unset, and the resource downloaded the artifacts to the same directory as the metadata files, preserving the existing behavior. When I tested my changes with asset_dir set to true, the artifacts ended up in a new subdirectory as expected.

Thank you for reading my pull-request!

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.

1 participant