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

refactor(init_client): merge InitClient and NewClient functions #765

Merged
merged 1 commit into from
Jan 25, 2024

Conversation

idrissneumann
Copy link
Contributor

What type of PR is this?

/kind cleanup

Any specific area of the project related to this PR?

/area outputs

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #763

@poiana poiana added the size/S label Jan 24, 2024
@Issif Issif added this to the 2.29.0 milestone Jan 24, 2024
@Issif Issif changed the title refactor(init_client): use InitClient instead of NewClient function wip: refactor(init_client): use InitClient instead of NewClient function Jan 24, 2024
@idrissneumann idrissneumann force-pushed the refactor_init_client branch 2 times, most recently from 8b422f1 to ee06084 Compare January 24, 2024 19:35
@poiana poiana added size/M and removed size/S labels Jan 24, 2024
@idrissneumann idrissneumann force-pushed the refactor_init_client branch 7 times, most recently from c589e21 to 134f48a Compare January 24, 2024 19:53
@poiana poiana added size/L and removed size/M labels Jan 24, 2024
@idrissneumann idrissneumann changed the title wip: refactor(init_client): use InitClient instead of NewClient function wip: refactor(init_client): merge InitClient and NewClient functions Jan 24, 2024
@idrissneumann idrissneumann force-pushed the refactor_init_client branch 5 times, most recently from 44ae4bc to f9dab62 Compare January 24, 2024 20:14
@idrissneumann idrissneumann marked this pull request as ready for review January 24, 2024 20:26
@idrissneumann idrissneumann changed the title wip: refactor(init_client): merge InitClient and NewClient functions refactor(init_client): merge InitClient and NewClient functions Jan 24, 2024
@idrissneumann
Copy link
Contributor Author

I think it's ready ^^ (cc @Issif )

Test with some outputs and docker ok

@poiana
Copy link

poiana commented Jan 25, 2024

LGTM label has been added.

Git tree hash: 10c6ea59ffa8da833e8566e557727f518d898d12

@poiana
Copy link

poiana commented Jan 25, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: idrissneumann, Issif

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@poiana poiana merged commit ab62637 into falcosecurity:master Jan 25, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Use the InitClient instead of NewClient function in all the outputs
3 participants