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

docs: Updates to the change log and readme files for upcoming patch release #677

Conversation

ritz303
Copy link
Contributor

@ritz303 ritz303 commented Jan 26, 2024

I've updated the CHANGELOG.md and README.md files for the upcoming patch release.

@ritz303 ritz303 marked this pull request as draft January 26, 2024 17:46
@ritz303 ritz303 changed the title Updates to the change log and readme files for upcoming patch release docs: Updates to the change log and readme files for upcoming patch release Jan 26, 2024
README.md Outdated
If you had a custom file path set for `redHatDependencyAnalyticsReportFilePath`, then you need to add your custom file path to the `reportFilePath` setting.

**Inline Vulnerability Severity Alerts** :
<br >You can set the vulnerability severity alert level to receive inline notifications for just errors or warnings.
Copy link
Collaborator

Choose a reason for hiding this comment

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

This sentence might not be very clear, the notifications will remain the same no matter the setting, but the sentence should make it clear that you have the option to configure whether these notifications appear as errors or warnings through the settings.

Copy link
Contributor Author

@ritz303 ritz303 Jan 29, 2024

Choose a reason for hiding this comment

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

Thanks for the feedback! I did a slight rewrite of the sentence, please see my latest commit. I hope that is clearer.

README.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@ritz303 ritz303 marked this pull request as ready for review January 30, 2024 19:51
@IlonaShishov
Copy link
Collaborator

/lgtm

@IlonaShishov IlonaShishov merged commit cba8b1b into fabric8-analytics:chore/issue-handling Feb 4, 2024
1 check passed
IlonaShishov added a commit that referenced this pull request Feb 5, 2024
* feat: add vulnerability alert severity setting (#674)

Signed-off-by: Ilona Shishov <[email protected]>

* chore: cleanup package.json (#675)

Signed-off-by: Ilona Shishov <[email protected]>

* fix: updrade to JS API version supporting pom modules for CA and spaces in file paths

Signed-off-by: Ilona Shishov <[email protected]>

* docs: Updates to the change log and readme files for upcoming patch release (#677)

* 1/26/24 - ritz303 : Updated the change log and readme files for upcoming patch release

* 1/29/24 - ritz303 : Edits from SME review

---------

Co-authored-by: Aron Gunn <[email protected]>

* docs: update changelog and README

Signed-off-by: Ilona Shishov <[email protected]>

---------

Signed-off-by: Ilona Shishov <[email protected]>
Co-authored-by: ritz303 <[email protected]>
Co-authored-by: Aron Gunn <[email protected]>
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.

3 participants