search

huynguyen-and-friend-projects / smoldb

Simple database in C
1 stars 3 forks source link

Format, lint, and new lint script #20

Closed nguyenhuy0905 closed 1 month ago

nguyenhuy0905 commented 1 month ago

Format, lint and new linting script

Formatted all current source file. Update linting script to exclude build dir and include test dir. Also included 2 new GitHub actions, CMake multi-platform build and CodeQL.

Format code so that it follows the format and linting options listed in .clang-format and .clang-tidy files.

I like the green tick effect when all GitHub actions run correctly :). To be fair, it isn't very useful for a project of this size.

I simply run formatting for all scripts, then go to each one and modify accordingly to the linter.

I added the GitHub action scripts, using the default provided by GitHub.

@qu-ngx @BAOELIETRAN @AbstractionHLR @bebeminhminh @aqt169

Pull request by: Huy Nguyen

nguyenhuy0905 commented 1 month ago

Oh yea, I forgot that this was on pull-request and made some random pushes there.

nguyenhuy0905 commented 1 month ago

Currently the check will fail when running MSVC though. Lemme cherry-pick the CMakeLists from the branch I fixed it.

nguyenhuy0905 commented 1 month ago

Nevermind, still failing.

nguyenhuy0905 commented 1 month ago

But it should be fixed after this PR. I have another giant PR coming.