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

[new release] syslog-message (1.0.0) #12809

Merged
merged 2 commits into from
Oct 16, 2018
Merged

[new release] syslog-message (1.0.0) #12809

merged 2 commits into from
Oct 16, 2018

Conversation

verbosemode
Copy link
Contributor

Syslog message parser

CHANGES:

CHANGES:

* Warning: encode function no longer truncates messages to 1024 bytes by default
* split message part into tag and content (verbosemode/syslog-message#20, by @hannesm)
* use result types instead of option (verbosemode/syslog-message#20, by @hannesm)
* remove transport-dependent length check from encode (verbosemode/syslog-message#20, by @hannesm)
* switch build system to Dune (verbosemode/syslog-message#19, by @dra27)
* add encode_local for sending to local syslog (verbosemode/syslog-message#17, by @dra27)
* forgot to thank @hannesm, @Leonidas-from-XIV for past contributions
@camelus
Copy link
Contributor

camelus commented Oct 15, 2018

☀️ All lint checks passed 3572a7c
  • These packages passed lint tests: syslog-message.1.0.0

☀️ Installability check (9791 → 9792)
  • new installable packages (1): syslog-message.1.0.0

@kit-ty-kate
Copy link
Member

Could you replicate the fix to upstream?

@kit-ty-kate
Copy link
Member

Thanks!

@kit-ty-kate kit-ty-kate merged commit 3e5ac5b into ocaml:master Oct 16, 2018
@verbosemode
Copy link
Contributor Author

Could you replicate the fix to upstream?

Thanks @kit-ty-kate . I'll fix it upstream. I figured that I also need to update some other fields upstream like dev-repo as well, to be opam 2.0 compatible.

@verbosemode verbosemode deleted the release-syslog-message-1.0.0 branch October 17, 2018 17:48
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