Skip to content

Latest commit

 

History

History
51 lines (34 loc) · 1.57 KB

LEARNING.md

File metadata and controls

51 lines (34 loc) · 1.57 KB

Note: Always check the source code close to the commit where this learning was introduced. Things might have changed in the future after that commit.

Name of the library is case-sensitive

2022-05-29 17:24:02 AGB

target_link_libraries(text-tests
        PRIVATE
        prod_lib
        CppUTest #1
        )
FetchContent_Declare(
        cpputest #2
        GIT_REPOSITORY [email protected]:cpputest/cpputest.git
        GIT_TAG        v4.0
        GIT_SHALLOW    1
)

(1): This name is case-sensitive. It's not the name defined in (2). It is the name defined in the resulting CMake build:

➜  gilded-rose-kata-c git:(master) ✗ find . -iname "libCppUTest.a"
./cmake-build-debug/_deps/cpputest-build/src/CppUTest/libCppUTest.a

To make it case-insensitive (or use any other name), use CMake's find_library + NAMES parameter (reference)

Untested code gets rusty

2022-05-29 19:56:57 AGB

By not using the original Makefile, run_once.sh, etc, it was not being tested.

I wanted to leave it for compatibility with the previous repo, but it got rusty of not being used.

Better to remove.

Quick to commit means all tools are sharp

2022-05-29 20:00:37 AGB

The speed to make commits is the bottleneck from thinking-writing-execution.

If there's a high speed, means that the tools are sharp:

  • Can't commit quickly if the build takes long
  • Can't commit quickly if tests are not automated + good coverage
  • Can't commit quickly if the process of committing itself is slow (e.g., GUI, waiting)