-
Notifications
You must be signed in to change notification settings - Fork 779
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
multiple paths? #55
Comments
Can anyone comment on if this is currently possible? |
With
Officially, there is no way to get multiple inputs at once from YAML (something like While we wait for official support, we can use a hack (similar to what |
It would be a very big improvement to support path like it is demonstrated here: I'm particularly interested in the latter ones, that could look like this: path:
- /first/path/file.ext
- /second/path/file.ext # take this one if the first is not found |
What will also be nice is to specify glob patterns too. So something like:
|
Now that #3 is published (:tada:), I guess this is the main remaining feature requested/proposed in #3 (comment):
|
To make it clear, you can already upload from multiple paths with The
PS: |
not quite @zhangyoufu If all the files are in the same working dir then it will work, however if you have a mix it will fail with Multiple search paths start to come up and that messes with the Line 46 in 97b7dac
|
… be quite useless. actions/upload-artifact#55 for details.
can we do this yet or not?
|
Hi @brandonros! As PR#94 has been created 13 hours ago and merged 5 hours ago, now the multiple paths can be done (in yml/yaml file) with Thank @konradpabjan so much for the work! Appreciate! |
Done! 😃 I've created a new release and updated the https://github.com/actions/upload-artifact#upload-using-multiple-paths-and-exclusions |
@konradpabjan thanks! It works nice! Regarding #55 (comment) (i.e. uploading multiple paths with different names in a single call to the action), shall I open a new issue? |
@eine To be frank, given that we can only accept input via key value pairs, doing something with multiple If we had support for list inputs with actions/toolkit#184, maybe we could do something like
but I'm not too big of a fan of this. I think it would be simpler for users to just call |
@konradpabjan I'd propose some other syntax that works around the key-value limitation. For example (removing the need for - uses: actions/upload-artifact@v2
with:
path: |
artifact1: path/to/my_artifact.tar.gz
artifact2: path/to/rpms/*.rpm, path/to/debs/*.deb
artifact3: "README.md", path/to/docs, !path/to/docs/tmp
"artifact:subname": path/to/another.tar.gz Although I understand that calling the action multiple times is an acceptable workaround given the constraints, I think that the UX should not be deteriorated because of incomplete implementations. Of course, it would make sense to delay such change to v3. |
Doesn't seems to work when uploading files from the different directories. E.g.
Issue:
|
Finally, which wildcard syntax could I use for multiple packages upload? I have multiple NuGet packages in several directories, and I do not want to explicitly specify the path for each packet. Current workflow yml:
|
so 2022 came and went any update on this? |
Hi, can i upload all files from two folder into root of result artifact? |
is multiple paths supported?
The text was updated successfully, but these errors were encountered: