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

Improvements suggested in the review #148

Conversation

nbertoldo
Copy link
Member

Related issue
#17

Description

This PR includes several corrections and improvements requested by the team in the revision of #45.

Configuration options

Logs/Alerts example

Tests

  • Compilation without warnings in every supported platform
    • Linux
    • Windows
    • MAC OS X

@nbertoldo nbertoldo self-assigned this Sep 11, 2024
@cborla cborla deleted the branch enhancement/17-migrate-syscollector-to-inventory-at-new-agent September 12, 2024 02:14
@cborla cborla closed this Sep 12, 2024
@nbertoldo nbertoldo reopened this Sep 12, 2024
@nbertoldo nbertoldo force-pushed the enhancement/17-improvements-suggested-in-the-review branch 3 times, most recently from 568c5d9 to 19af211 Compare September 12, 2024 23:15
@nbertoldo nbertoldo force-pushed the enhancement/17-improvements-suggested-in-the-review branch 4 times, most recently from 380c84d to 4c19f2f Compare September 20, 2024 20:30
@nbertoldo nbertoldo marked this pull request as ready for review September 20, 2024 20:36
@LucioDonda LucioDonda force-pushed the enhancement/17-migrate-syscollector-to-inventory-at-new-agent branch from b8fe691 to e8c737f Compare September 20, 2024 20:52
@nbertoldo nbertoldo force-pushed the enhancement/17-improvements-suggested-in-the-review branch from 4c19f2f to a72f7db Compare September 23, 2024 20:26
@cborla cborla merged commit 4537aa5 into enhancement/17-migrate-syscollector-to-inventory-at-new-agent Sep 23, 2024
1 of 5 checks passed
@cborla cborla deleted the enhancement/17-improvements-suggested-in-the-review branch September 23, 2024 20:38
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