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

chore(deps): update actions/upload-artifact action to v4 #6569

Merged
merged 4 commits into from May 14, 2024

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented May 9, 2024

Mend Renovate

This PR contains the following updates:

Package Type Update Change
actions/upload-artifact action major v3 -> v4

Release Notes

actions/upload-artifact (actions/upload-artifact)

v4

Compare Source


Configuration

📅 Schedule: Branch creation - "before 4am on Monday" (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate. View repository job log here.

@renovate renovate bot requested a review from a team as a code owner May 9, 2024 13:49
@renovate renovate bot added the dependencies Pull requests that update a dependency file label May 9, 2024
Copy link

github-actions bot commented May 9, 2024

📊 Benchmark results

Comparing with b2d35aa

  • Dependency count: 1,341 (no change)
  • Package size: 311 MB (no change)
  • Number of ts-expect-error directives: 993 (no change)

kodiakhq[bot]
kodiakhq bot previously approved these changes May 9, 2024
@renovate renovate bot force-pushed the renovate/major-github-artifact-actions branch from dcc9148 to 3ae9fec Compare May 9, 2024 14:17
Copy link
Contributor Author

renovate bot commented May 9, 2024

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

kodiakhq[bot]
kodiakhq bot previously approved these changes May 9, 2024
kodiakhq[bot]
kodiakhq bot previously approved these changes May 9, 2024
kodiakhq[bot]
kodiakhq bot previously approved these changes May 9, 2024
kodiakhq[bot]
kodiakhq bot previously approved these changes May 9, 2024
kodiakhq[bot]
kodiakhq bot previously approved these changes May 9, 2024
kodiakhq[bot]
kodiakhq bot previously approved these changes May 9, 2024
kodiakhq[bot]
kodiakhq bot previously approved these changes May 9, 2024
@mrstork mrstork removed the request for review from a team May 9, 2024 18:56
@mrstork mrstork force-pushed the renovate/major-github-artifact-actions branch from 56b84d0 to a4134fe Compare May 9, 2024 19:00
kodiakhq[bot]
kodiakhq bot previously approved these changes May 9, 2024
kodiakhq[bot]
kodiakhq bot previously approved these changes May 9, 2024
kodiakhq[bot]
kodiakhq bot previously approved these changes May 9, 2024

- name: Sanitize shard for artefact name
id: sanitize-shard-name
run: echo "shard=$(echo '${{ matrix.shard }}' | tr '/' '-')" >> $GITHUB_OUTPUT
Copy link
Contributor

Choose a reason for hiding this comment

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

GitHub doesn't allow / in the file name (see L116)

path: |
/home/runner/.npm/_logs/**/*

- uses: codecov/codecov-action@v4
continue-on-error: true
with:
flags: ${{ steps.test-coverage-flags.outputs.os }},${{ steps.test-coverage-flags.outputs.node }}
token: ${{ secrets.CODECOV_TOKEN }}
Copy link
Contributor

@mrstork mrstork May 10, 2024

Choose a reason for hiding this comment

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

This is currently not being picked up, could we make sure it hasn't accidentally been removed? (I don't have permissions to view)

Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think this secret has ever existed - I think this PR is trying to tell us that we need one now with the newest version?

Copy link
Contributor

Choose a reason for hiding this comment

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

Do we have an account for https://app.codecov.io/gh/netlify/cli? Can we retrieve a token from that account? Otherwise I can downgrade us back to codecov/codecov-action@v3 (where the token isn't required for the workflow step). There might be a case to be made for removing the codecov step completely as it doesn't seem to be very depended on, though you folks would have more context on that topic.

@mrstork mrstork merged commit 920f16e into main May 14, 2024
48 checks passed
@mrstork mrstork deleted the renovate/major-github-artifact-actions branch May 14, 2024 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants