Fix 'jf' path on cleanup and validate JobSummary supported CLI version #201

sverdlov93 · 2024-09-03T14:56:25Z

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 #200

…o fix-path-cache

…o fix-path-cache # Conflicts: # lib/utils.js # src/utils.ts

Co-authored-by: Yahav Itschak <yahavi@users.noreply.github.com>

src/cleanup.ts

src/utils.ts

src/cleanup.ts

Co-authored-by: Yahav Itschak <yahavi@users.noreply.github.com>

sverdlov93 added 5 commits September 2, 2024 17:52

Fix path

1f707ff

Update to version 4.3.2

36f0d5f

Merge branch 'master' of https://github.com/jfrog/setup-jfrog-cli int…

14b297c

…o fix-path-cache

Merge branch 'master' of https://github.com/jfrog/setup-jfrog-cli int…

6c9b253

…o fix-path-cache

Update to version 4.3.3

3fa8574

sverdlov93 had a problem deploying to frogbot September 3, 2024 14:56 — with GitHub Actions Failure

sverdlov93 requested a review from yahavi September 3, 2024 14:56

sverdlov93 added the safe to test Approve running integration tests on a pull request label Sep 3, 2024

github-actions bot removed the safe to test Approve running integration tests on a pull request label Sep 3, 2024

Update to version 4.3.3

6aac057

sverdlov93 had a problem deploying to frogbot September 3, 2024 15:08 — with GitHub Actions Failure

sverdlov93 added the safe to test Approve running integration tests on a pull request label Sep 3, 2024

github-actions bot removed the safe to test Approve running integration tests on a pull request label Sep 3, 2024

Update to version 4.3.3

a8d9909

sverdlov93 had a problem deploying to frogbot September 3, 2024 15:13 — with GitHub Actions Failure

sverdlov93 added the safe to test Approve running integration tests on a pull request label Sep 3, 2024

github-actions bot removed the safe to test Approve running integration tests on a pull request label Sep 3, 2024

Update to version 4.3.3

45dac24

sverdlov93 had a problem deploying to frogbot September 3, 2024 15:22 — with GitHub Actions Failure

Update to version 4.3.3

bb23156

sverdlov93 had a problem deploying to frogbot September 3, 2024 15:53 — with GitHub Actions Failure

Update to version 4.3.3

c467e4c

sverdlov93 had a problem deploying to frogbot September 3, 2024 15:55 — with GitHub Actions Failure

sverdlov93 added the safe to test Approve running integration tests on a pull request label Sep 3, 2024

github-actions bot removed the safe to test Approve running integration tests on a pull request label Sep 3, 2024

sverdlov93 mentioned this pull request Sep 3, 2024

Cleanup error: Unable to find jf executable #200

Closed

Update to version 4.3.3

bed193c

sverdlov93 had a problem deploying to frogbot September 3, 2024 16:19 — with GitHub Actions Failure

sverdlov93 added 2 commits September 3, 2024 19:20

Merge branch 'master' of https://github.com/jfrog/setup-jfrog-cli int…

33a0a39

…o fix-path-cache # Conflicts: # lib/utils.js # src/utils.ts

Update to version 4.3.3

ea46492

Update to version 4.3.3

36b9d45

sverdlov93 had a problem deploying to frogbot September 4, 2024 09:11 — with GitHub Actions Failure

Update to version 4.3.3

a197588

sverdlov93 had a problem deploying to frogbot September 4, 2024 11:07 — with GitHub Actions Failure

Update to version 4.3.3

107905e

sverdlov93 had a problem deploying to frogbot September 4, 2024 11:12 — with GitHub Actions Failure

Update to version 4.3.

1dc2982

sverdlov93 had a problem deploying to frogbot September 4, 2024 11:15 — with GitHub Actions Failure

Update to version 4.3.

dc9e306

sverdlov93 had a problem deploying to frogbot September 4, 2024 12:45 — with GitHub Actions Failure

Update to version 4.3.

0d66af4

sverdlov93 had a problem deploying to frogbot September 4, 2024 13:02 — with GitHub Actions Failure

sverdlov93 requested a review from yahavi September 4, 2024 13:02

sverdlov93 changed the title ~~Fix missing 'jf' path on step cleanup~~ Fix missing 'jf' path on step cleanup and validate JobSummary supported CLI version Sep 4, 2024

sverdlov93 changed the title ~~Fix missing 'jf' path on step cleanup and validate JobSummary supported CLI version~~ Fix 'jf' path on cleanup and validate JobSummary supported CLI version Sep 4, 2024

Update to version 4.3.

171557c

sverdlov93 had a problem deploying to frogbot September 4, 2024 13:24 — with GitHub Actions Failure

Update .github/workflows/auto-build-publish-test.yml

f1087f2

Co-authored-by: Yahav Itschak <yahavi@users.noreply.github.com>

sverdlov93 had a problem deploying to frogbot September 4, 2024 13:26 — with GitHub Actions Failure

yahavi requested changes Sep 4, 2024

View reviewed changes

src/cleanup.ts Show resolved Hide resolved

src/utils.ts Show resolved Hide resolved

src/utils.ts Show resolved Hide resolved

src/utils.ts Outdated Show resolved Hide resolved

src/cleanup.ts Outdated Show resolved Hide resolved

src/cleanup.ts Show resolved Hide resolved

Update src/cleanup.ts

e7517c9

Co-authored-by: Yahav Itschak <yahavi@users.noreply.github.com>

RobiNino had a problem deploying to frogbot September 4, 2024 15:57 — with GitHub Actions Failure

Update src/utils.ts

769af07

Co-authored-by: Yahav Itschak <yahavi@users.noreply.github.com>

RobiNino had a problem deploying to frogbot September 4, 2024 15:57 — with GitHub Actions Failure

RobiNino added 2 commits September 4, 2024 19:33

CR comments

4a7729b

CR comments

a29a2bd

RobiNino had a problem deploying to frogbot September 4, 2024 16:34 — with GitHub Actions Failure

RobiNino added the bug Something isn't working label Sep 4, 2024

RobiNino merged commit c10541d into master Sep 4, 2024
22 of 23 checks passed

jrbushell mentioned this pull request Sep 6, 2024

EROFS read-only file system error when jfrog binary is already cached #211

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix 'jf' path on cleanup and validate JobSummary supported CLI version #201

Fix 'jf' path on cleanup and validate JobSummary supported CLI version #201

sverdlov93 commented Sep 3, 2024

Fix 'jf' path on cleanup and validate JobSummary supported CLI version #201

Fix 'jf' path on cleanup and validate JobSummary supported CLI version #201

Conversation

sverdlov93 commented Sep 3, 2024