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

Compare across forks #707

Closed
wants to merge 8 commits into from
Closed

Compare across forks #707

wants to merge 8 commits into from

Conversation

moneyglitxh
Copy link

New methods

aiqiaoy and others added 8 commits December 10, 2020 16:00
* actions/artifact 0.4.2

* Update releases note
* Retry all http calls for artifact upload and download

* Extra debug information

* Fix lint

* Always read response body

* PR Feedback

* Change error message if patch call fails

* Add exponential backoff when retrying

* Rework tests and add diagnostic info if exception thrown

* Fix lint

* fix lint error for real this time

* PR cleanup

* 0.5.0 @actions/artifact release

* Display diagnostic info if non-retryable code is hit
- Print cache size when saving cache similarly to restoring
- Print restore success similarly to saving
- Print cached file list if debug logging is enabled

See also: actions/cache#471
From February 2021, in order to provide feedback on pull requests, Code Scanning workflows must be configured with both `push` and `pull_request` triggers. This is because Code Scanning compares the results from a pull request against the results for the base branch to tell you only what has changed between the two.

Early in the beta period we supported displaying results on pull requests for workflows with only `push` triggers, but have discontinued support as this proved to be less robust.

See https://docs.github.com/en/free-pro-team@latest/github/finding-security-vulnerabilities-and-errors-in-your-code/configuring-code-scanning#scanning-pull-requests for more information on how best to configure your Code Scanning workflows.
Make caching more verbose
* use GNU tar on macOS

* remove unnecessary code

* formatting fix

* fix tests

* fix more tests

* typo fix

* fix test
@moneyglitxh moneyglitxh requested a review from a team as a code owner February 3, 2021 20:52
@moneyglitxh moneyglitxh requested a review from a team February 3, 2021 20:52
@moneyglitxh
Copy link
Author

Think I got it

Copy link
Author

@moneyglitxh moneyglitxh left a comment

Choose a reason for hiding this comment

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

Finished

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
*off-topic This issue is not related to this project or is not actionable
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants