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

Add Cache Hit Output #373

Merged
merged 1 commit into from Sep 7, 2022

Conversation

ansraliant
Copy link
Contributor

Description:

Cache hit was being used as stated in this pr #275.
However, it was not included in the outputs of the action. It did not cause any mayor issues, but if you were to lint the workflows, then an output not defined error would pop up.
This Commit just adds the output to the list. Nothing more

Related issue:
#275
#306

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

Cache hit was being used as stated in this pr
actions#275.
However, it was not included in the outputs of the action. It did not
cause any mayor issues, but if you were to lint the workflows, then an
output not defined error would pop up.
This Commit just adds the output to the list. Nothing more
@ansraliant ansraliant requested a review from a team Aug 23, 2022
@dmitry-shibanov dmitry-shibanov merged commit 749e449 into actions:main Sep 7, 2022
169 checks passed
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.

None yet

3 participants