-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
seq: add recipe #25182
base: master
Are you sure you want to change the base?
seq: add recipe #25182
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit 1afd704seq/1.1.0@#e6174c21e653105d9b7d7d599cdba07d
|
Conan v1 pipeline ✔️All green in build 5 (
Conan v2 pipeline ✔️
All green in build 5 (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Some comments, looks good for the most part :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The recipe looks good, but the CI did not generate packages for header_only=False. Need to check the CI, blocking to avoid merging without the missing configuration.
Hi @toge we have a fix ready for this, but we're holding the update for a bit. You don't need to do anything else for this PR, we'll retrigger it once the issue has been fixed on our side, thanks for your patience! :) |
Summary
Changes to recipe: seq/1.1.0
Motivation
Seq is a stable container library.
Details
https://github.com/Thermadiag/seq/