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.

Sign up for GitHub

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

Jump to bottom

Add update_check config reference #6673

Merged
merged 4 commits into from
Oct 27, 2023
Merged

Add update_check config reference #6673

merged 4 commits into from
Oct 27, 2023

Conversation

nico-stefani
Copy link
Member

@nico-stefani nico-stefani commented Oct 23, 2023 •
edited
Loading

Description

This PR closes #6641. Adds reference for the new configuration (update_check) in the global section of the ossec.conf.

Checks

Docs building

  • Compiles without warnings.

Code formatting and web optimization

  • Uses three spaces indentation.
  • Adds or updates meta descriptions accordingly.
  • Updates the redirects.js script if necessary (check this guide).

Writing style

  • Uses present tense, active voice, and semi-formal registry.
  • Uses short, simple sentences.
  • Uses bold for user interface elements, italics for key terms or emphasis, and code font for Bash commands, file names, REST paths, and code.

@nico-stefani nico-stefani self-assigned this Oct 23, 2023
@nico-stefani nico-stefani linked an issue Oct 23, 2023 that may be closed by this pull request
Add documentation for the update check system #6641
Closed
Base automatically changed from 4.8.0-update-release-notes to 4.8.0 October 23, 2023 20:07
@fdalmaup fdalmaup self-requested a review October 24, 2023 12:27
@nico-stefani @fdalmaup
Apply suggestions from code review
21ab1ee 
Co-authored-by: Facundo Dalmau <facundo.dalmau@wazuh.com>
@nico-stefani nico-stefani requested a review from fdalmaup October 24, 2023 13:56
fdalmaup
fdalmaup approved these changes Oct 24, 2023
View reviewed changes
Copy link
Member

@fdalmaup fdalmaup left a comment

Choose a reason for hiding this comment

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

LGTM!

@nico-stefani @Selutario
Apply suggestions from code review
8d9260d 
Co-authored-by: José Luis López Sánchez <joseluis.lopez@wazuh.com>
@javimed javimed added type/enhancement Enhancement issue level/task Task issue labels Oct 26, 2023
@nico-stefani @javimed
Apply suggestions from code review
2d53cf8 
Co-authored-by: Javier Medeot <javier.medeot@wazuh.com>
@javimed javimed merged commit a14c6be into 4.8.0 Oct 27, 2023
@javimed javimed deleted the feature/6641-add-documentation-for-the-update-check-system branch October 27, 2023 12:56
@Selutario Selutario mentioned this pull request Oct 30, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@javimed javimed javimed approved these changes

@fdalmaup fdalmaup fdalmaup approved these changes

@Selutario Selutario Selutario approved these changes

Assignees

@nico-stefani nico-stefani

Labels
level/task Task issue type/enhancement Enhancement issue
Projects
No open projects
Release 4.8.0
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add documentation for the update check system
4 participants
@nico-stefani @Selutario @fdalmaup @javimed