Skip to content

Лагунов Евгений Сергеевич#31

Open
jonyks wants to merge 3 commits intocppdevcourse:masterfrom
jonyks:master
Open

Лагунов Евгений Сергеевич#31
jonyks wants to merge 3 commits intocppdevcourse:masterfrom
jonyks:master

Conversation

@jonyks
Copy link
Copy Markdown

@jonyks jonyks commented Dec 6, 2024

No description provided.

Copy link
Copy Markdown
Collaborator

@czertyaka czertyaka left a comment

Choose a reason for hiding this comment

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

Необходимо убрать директорию build из Git

Comment thread CMakeLists.txt
find_package(GTest REQUIRED)
add_executable(two-sum-test "${TEST_DIR}/test.cpp" "${SRC_DIR}/two-sum.cpp")
target_link_libraries(two-sum-test PRIVATE gtest::gtest)
target_link_libraries(two-sum-test PRIVATE GTest::GTest GTest::Main)
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Зачем потребовалось изменение? Не компилировалось?

Copy link
Copy Markdown
Author

@jonyks jonyks Dec 8, 2024

Choose a reason for hiding this comment

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

Верно, вызывалась ошибка указывающая на неправильную строку кода
Без имени2

Comment thread src/two-sum.hpp Outdated
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Зачем потребовалось копировать файл из include?

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.

2 participants