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

Support downloading multiple artifacts #184

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

diogokiss
Copy link

No description provided.

In order to support multiline inputs we need to upgrade the '@actions/core'
NPM package to version v1.10.0.
It's smart to keep the NodeJS version pinned so we can run all build jobs
consistently.

Otherwise, we might run into some versions mismatch like below:
```
npm ERR! While resolving: @typescript-eslint/parser@2.30.0
npm ERR! Found: eslint@7.4.0
npm ERR! node_modules/eslint
npm ERR!   dev eslint@"^7.4.0" from the root project
npm ERR!   peer eslint@"^5.0.0 || ^6.0.0 || ^7.0.0" from @typescript-eslint/eslint-plugin@4.11.1
npm ERR!   node_modules/@typescript-eslint/eslint-plugin
npm ERR! @typescript-eslint/eslint-plugin@">=2.25.0" from eslint-plugin-github@4.1.1
npm ERR! node_modules/eslint-plugin-github
npm ERR!   dev eslint-plugin-github@"^4.1.1" from the root project
npm ERR!   7 more (@typescript-eslint/experimental-utils, ...)
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer eslint@"^5.0.0 || ^6.0.0" from @typescript-eslint/parser@2.30.0
npm ERR! node_modules/@typescript-eslint/parser
npm ERR!   dev @typescript-eslint/parser@"^2.30.0" from the root project
npm ERR!   @typescript-eslint/parser@">=2.25.0" from eslint-plugin-github@4.1.1
npm ERR!   node_modules/eslint-plugin-github
npm ERR! dev eslint-plugin-github@"^4.1.1" from the root project
npm ERR!
npm ERR! Conflicting peer dependency: eslint@6.8.0
npm ERR! node_modules/eslint
npm ERR!   peer eslint@"^5.0.0 || ^6.0.0" from @typescript-eslint/parser@2.30.0
npm ERR!   node_modules/@typescript-eslint/parser
npm ERR! dev @typescript-eslint/parser@"^2.30.0" from the root project
npm ERR! @typescript-eslint/parser@">=2.25.0" from eslint-plugin-github@4.1.1
npm ERR! node_modules/eslint-plugin-github
npm ERR!   dev eslint-plugin-github@"^4.1.1" from the root project
```
There is a few use cases where we want to download a few different artifacts but
not all of them. This commit implements this support, without breaking backward
compatibility. Two build test cases were also added to the pipeline.
@diogokiss diogokiss requested a review from a team as a code owner October 13, 2022 11:55
Copy link

@mattheo010630 mattheo010630 left a comment

Choose a reason for hiding this comment

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

Lates

@Truly93
Copy link

Truly93 commented Dec 8, 2023

steps:

  • uses: actions/checkout@v3

  • uses: actions/download-artifact@v3

  • name: Display structure of downloaded files
    run: ls -R[steps:

  • uses: actions/checkout@v3

  • uses: actions/download-artifact@v3
    id: download
    with:
    name: 'my-artifact'
    path: path/to/artifacts

  • name: 'Echo download path'
    run: echo ${{steps.download.outputs.download-path}}](url)

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.

5 participants