Skip to content

feat: reexport broadcast and native_tls from fasyslog#81

Merged
tisonkun merged 2 commits intofast:mainfrom
BBArikL:main
Nov 17, 2024
Merged

feat: reexport broadcast and native_tls from fasyslog#81
tisonkun merged 2 commits intofast:mainfrom
BBArikL:main

Conversation

@BBArikL
Copy link
Copy Markdown
Contributor

@BBArikL BBArikL commented Nov 14, 2024

Reexport BroadcastSender from fast/fasyslog#6

@tisonkun
Copy link
Copy Markdown
Contributor

Thank you! I'll handle this patch today.

Signed-off-by: tison <wander4096@gmail.com>
@tisonkun tisonkun changed the title feat: reexport BroadcastSender from fasyslog#6 feat: reexport broadcast and native_tls from fasyslog Nov 17, 2024
@tisonkun
Copy link
Copy Markdown
Contributor

@BBArikL Pending for merging ...

You can ping me the next time, since PR doesn't send a direct notification, instead, it only sends to subscribed@noreply.github.com.

Besides, I'd recommend you to use a different branch name (rather than main) for contributing. Otherwise, when using git push to modify the patch, there is some frictions like "The upstream branch of your current branch does not match the name of your current branch."

Copy link
Copy Markdown
Contributor

@tisonkun tisonkun left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution!

@tisonkun tisonkun merged commit ed0f139 into fast:main Nov 17, 2024
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