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

[backport/release/2.11] build: add CMake configuration for evread module #9874

Conversation

mkokryashkin
Copy link
Contributor

This patch adds necessary cmake configurations for the <evread.lua> module, so it can be used later to implement human-readable error reporting in profile parsers.

Part of #9217

NO_DOC=LuaJIT submodule
NO_TEST=covered by the LuaJIT tests
NO_CHANGELOG=build

(cherry picked from commit e01fe8f)

This patch adds necessary cmake configurations for the
<evread.lua> module, so it can be used later to implement
human-readable error reporting in profile parsers.

Part of tarantool#9217

NO_DOC=LuaJIT submodule
NO_TEST=covered by the LuaJIT tests
NO_CHANGELOG=build

(cherry picked from commit e01fe8f)
@mkokryashkin mkokryashkin added the full-ci Enables all tests for a pull request label Mar 26, 2024
@mkokryashkin mkokryashkin added full-ci Enables all tests for a pull request and removed full-ci Enables all tests for a pull request labels Mar 26, 2024
@coveralls
Copy link

Coverage Status

coverage: 85.814% (-0.001%) from 85.815%
when pulling 8f7c051 on mkokryashkin:mkokryashkin/profile-parsers-refactoring-2.11
into fe4a9e2
on tarantool:release/2.11
.

Copy link
Collaborator

@Buristan Buristan left a comment

Choose a reason for hiding this comment

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

Hi, Maxim!
Thanks for the patch!
LGTM!

@ylobankov ylobankov merged commit 8aad968 into tarantool:release/2.11 Mar 26, 2024
113 of 226 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
full-ci Enables all tests for a pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants