-
Notifications
You must be signed in to change notification settings - Fork 84
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
Release 1.2.0 #215
Comments
Hello, @luismartingil! As I see you would like to have release to Conan to be able to use jinja2cpp with new deps? Anything else we need to include into this release except closing current PR's and maybe one-two bugs? |
But let's move sequentially - release, then publication. |
It seems that fix of #213 is also good candidate for release |
@luismartingil , something like that https://github.com/jinja2cpp/Jinja2Cpp/releases/tag/1.2.0-beta ?=) |
@rmorozov Do you want us to check the beta release prior to making it official? |
@luismartingil , if it is possible, it would be great=) |
Ok give us some time, thanks! |
Looks good to me. I wasn't able to test it within our product, but the build, unit tests, and release content look good to me. @luismartingil fyi. |
@rmorozov Good to go for us! |
@rmorozov You're working on the latest details before the release, right? |
@luismartingil , I think this version could be promoted to release. And now I am working on replacing value-ptr with polymorphic value - #219 It seems that value-ptr is broken at master branch. |
I'll try to fire new release when I'll manage to overcome problems in #219 + maybe some fixes you find in 1.2.0/1.2.1. |
@luismartingil , as far as I understand know we need to make a PR to Conan-center-index? |
@rmorozov Available in Conan now... thanks! conan-io/conan-center-index#7147 |
@rmorozov
FYI @danielmartinezruiz @league55
The text was updated successfully, but these errors were encountered: