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

README: required programs for self-hosted and container #162

Merged
merged 2 commits into from
Dec 16, 2024

Conversation

trgalho
Copy link
Contributor

@trgalho trgalho commented Dec 11, 2024

Action is not working when running in a container curl and unzip are not installed.
This document the required programs you need to run the action properly in this context.

  • Please explain your motives to contribute this change: what problem you are trying to fix, what improvement you are trying to make
  • Make sure any code you changed is covered by tests
  • If there is a JIRA ticket available, please make your commits and pull request start with the ticket ID (SONAR-XXXX)

We will try to give you feedback on your contribution as quickly as possible.

Thank You!
The SonarSource Team

@antonioaversa antonioaversa self-requested a review December 11, 2024 17:36
@antonioaversa antonioaversa changed the title docs: required programs for self-hosted and container README: required programs for self-hosted and container Dec 11, 2024
Copy link
Contributor

@antonioaversa antonioaversa left a comment

Choose a reason for hiding this comment

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

Looks good to me, thank you for this contribution!

We will merge it close to the release of the upcoming v4.2.0.

@antonioaversa antonioaversa marked this pull request as draft December 11, 2024 18:22
@antonioaversa antonioaversa marked this pull request as ready for review December 16, 2024 09:29
@antonioaversa antonioaversa merged commit d1fd0eb into SonarSource:master Dec 16, 2024
51 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.

2 participants