Skip to content

Fix missing cached JFrog CLI path on cleanup task#197

Merged
sverdlov93 merged 1 commit intomasterfrom
fix-path-cache
Sep 2, 2024
Merged

Fix missing cached JFrog CLI path on cleanup task#197
sverdlov93 merged 1 commit intomasterfrom
fix-path-cache

Conversation

@sverdlov93
Copy link
Copy Markdown
Contributor

@sverdlov93 sverdlov93 commented Sep 2, 2024

  • All tests passed. If this feature is not already covered by the tests, I added new tests.
  • I used npm run format for formatting the code before submitting the pull request.

should fix #196

@sverdlov93 sverdlov93 requested a review from yahavi September 2, 2024 14:53
@sverdlov93 sverdlov93 added the bug Something isn't working label Sep 2, 2024
@sverdlov93 sverdlov93 added the safe to test Approve running integration tests on a pull request label Sep 2, 2024
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Sep 2, 2024
@sverdlov93 sverdlov93 merged commit 900773d into master Sep 2, 2024
if (jfExecDir && jfrogExecDir) {
core.addPath(jfExecDir);
core.addPath(jfrogExecDir);
// Save the JF CLI path to use on cleanup. saveState/getState are methods to pass data between a step, and it's cleanup function.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It's --> its

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cleanup Errors even with disable-auto-build-publish set to true

3 participants