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 option to disable Job Summaries #169

Merged
merged 16 commits into from
Jul 11, 2024
Merged

Conversation

EyalDelarea
Copy link
Contributor

@EyalDelarea EyalDelarea commented Jul 3, 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.

Adding feature flag for Job Summaries.

In order to disable the creation of job summaries, disable-job-summary attribute can be provided the the setup CLI. default value is false.

example:

- uses: jfrog/setup-jfrog-cli@v4
  with:
    disable-job-summary: true

Under the hood -

In order to avoid changes in the CLI itself, when the path of the output dir is not defined the CLI will ignore the recording of commands.
Therefore, when the feature flag is disabled it will just make sure to unset the output dir path.

@EyalDelarea EyalDelarea added the improvement Automatically generated release notes label Jul 3, 2024
@EyalDelarea EyalDelarea linked an issue Jul 3, 2024 that may be closed by this pull request
@EyalDelarea EyalDelarea requested a review from yahavi July 3, 2024 09:06
@EyalDelarea EyalDelarea changed the title Add Job Summaries feature flag Add option to disable Job Summaries Jul 8, 2024
action.yml Outdated Show resolved Hide resolved
src/utils.ts Outdated Show resolved Hide resolved
test/main.spec.ts Outdated Show resolved Hide resolved
test/main.spec.ts Outdated Show resolved Hide resolved
test/main.spec.ts Outdated Show resolved Hide resolved
action.yml Outdated Show resolved Hide resolved
action.yml Outdated Show resolved Hide resolved
@EyalDelarea EyalDelarea added the safe to test Approve running integration tests on a pull request label Jul 10, 2024
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Jul 10, 2024
@EyalDelarea EyalDelarea added the safe to test Approve running integration tests on a pull request label Jul 10, 2024
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Jul 10, 2024
@EyalDelarea EyalDelarea linked an issue Jul 10, 2024 that may be closed by this pull request
@EyalDelarea EyalDelarea requested a review from yahavi July 10, 2024 15:14
@EyalDelarea EyalDelarea added the safe to test Approve running integration tests on a pull request label Jul 11, 2024
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Jul 11, 2024
Copy link

👍 Frogbot scanned this pull request and did not find any new security issues.


@EyalDelarea EyalDelarea merged commit 8db8d2b into jfrog:master Jul 11, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Automatically generated release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Misleading Content in OIDC Example Add option to disable Summary output
2 participants