Skip to content

Conversation

@Yaraslaut
Copy link
Member

Add simple asyncronous test

@Yaraslaut Yaraslaut requested a review from a team as a code owner November 27, 2025 18:32
@Yaraslaut Yaraslaut changed the title add simple test using stdexec Add simple test that is using stdexec Nov 27, 2025
@Yaraslaut Yaraslaut force-pushed the tests/simple_test_using_stdexec branch 2 times, most recently from c92b2dc to 45be83e Compare November 27, 2025 20:07
@Yaraslaut Yaraslaut force-pushed the tests/simple_test_using_stdexec branch from 45be83e to f4a97c4 Compare November 27, 2025 20:38
@Yaraslaut Yaraslaut force-pushed the tests/simple_test_using_stdexec branch 2 times, most recently from 8ef21ae to a793559 Compare November 27, 2025 21:00
@Yaraslaut Yaraslaut force-pushed the tests/simple_test_using_stdexec branch 6 times, most recently from 5e29487 to 5e77ea1 Compare November 28, 2025 15:51
@Yaraslaut Yaraslaut force-pushed the tests/simple_test_using_stdexec branch 4 times, most recently from b26960d to f32362b Compare November 28, 2025 16:34
@Yaraslaut Yaraslaut force-pushed the tests/simple_test_using_stdexec branch from 93af413 to 7645a9d Compare November 28, 2025 16:46
@Yaraslaut Yaraslaut merged commit 06c2262 into master Nov 28, 2025
14 checks passed
@Yaraslaut Yaraslaut deleted the tests/simple_test_using_stdexec branch November 28, 2025 17:05
endif()
# if(${PEDANTIC_COMPILER_WERROR})
# try_add_compile_options(/WX)
# endif()
Copy link
Member

Choose a reason for hiding this comment

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

I'm not sure man, if PEDANTIC_COMPILER_WERROR is ON, then /WX should be set for MSVC. This is exactly that this compiler option is there for.

If we figure, that we can't compile (e.g. due to stdexcec) on MSVC, with PEDANTIC_COMPILER_WERROR=ON, then we should tweak CI to have that set to OFF when MSVC is used.


#include "../Utils.hpp"
#include "Entities.hpp"
#include "Lightweight/SqlStatement.hpp"
Copy link
Member

Choose a reason for hiding this comment

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

bad include :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants