-
Notifications
You must be signed in to change notification settings - Fork 129
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
MAINT: Bump actions/download-artifact from 3 to 4 #4096
MAINT: Bump actions/download-artifact from 3 to 4 #4096
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4096 +/- ##
===========================================
- Coverage 82.10% 31.00% -51.10%
===========================================
Files 183 183
Lines 64464 64464
===========================================
- Hits 52926 19987 -32939
- Misses 11538 44477 +32939 |
@MaxJPRey @SMoraisAnsys Not sure if we should update, please check if this could introduce any issue |
@Samuelopez-ansys we should avoid to accept this PR for the moment as we had to revert from v4 to v3 too avoid an issue with the number of file we are uploading (cf actions/upload-artifact#485) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Blocked until documentation build is fixed
cae6ea1
to
58fcc75
Compare
Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 3 to 4. - [Release notes](https://github.com/actions/download-artifact/releases) - [Commits](actions/download-artifact@v3...v4) --- updated-dependencies: - dependency-name: actions/download-artifact dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
58fcc75
to
801bf00
Compare
@SMoraisAnsys @MaxJPRey @maxcapodi78 What should we do with these Pull requests? Not sure if it is going to work, we have to check these 3 updates: ansys/actions/doc-deploy-stable@v5 |
If it bother you, we can close it but I would prefer to leave the 3 PR open as a reminder. If no fix is proposed soon, I'll try to find some time and look for a bugfix to propose. |
@SMoraisAnsys I fully agree with you. |
@SMoraisAnsys What should we do with this PR? |
@Samuelopez-ansys I'm on days off but I'll see if we can do the associated changes if the CI refactoring. |
Hello @maxcapodi78! This PR was created by dependabot and has not been updated in the last 30 days. Could you please review it? If you want this repository to be excluded from this automated process, please let us know by filling in the opt-out request form. |
OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting If you change your mind, just re-open this PR and I'll resolve any conflicts on it. |
Bumps actions/download-artifact from 3 to 4.
Release notes
Sourced from actions/download-artifact's releases.
Commits
6b208ae
Merge pull request #274 from actions/vmjoseph/timeout-patch6c5b580
only adding updated license5f5015d
readding index1fddaaf
Revert "updating licenses"8aa9e21
Revert "updating dist"657edd9
updating licenses555a2fc
updating dist4fc4d70
updating lock072ac9d
updating version no038dc03
updating version noYou can trigger a rebase of this PR by commenting
@dependabot rebase
.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebase
will rebase this PR@dependabot recreate
will recreate this PR, overwriting any edits that have been made to it@dependabot merge
will merge this PR after your CI passes on it@dependabot squash and merge
will squash and merge this PR after your CI passes on it@dependabot cancel merge
will cancel a previously requested merge and block automerging@dependabot reopen
will reopen this PR if it is closed@dependabot close
will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditions
will show all of the ignore conditions of the specified dependency@dependabot ignore this major version
will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor version
will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependency
will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)