Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces four new skill documentation files for the Firebase C++ SDK, covering native build workflows, Doxygen commenting standards, code formatting procedures, and integration test execution. The feedback focuses on correcting the test runner documentation, specifically clarifying the difference between building the SDK and building test apps for Desktop, ensuring manual CMake commands include the necessary SDK directory flag, and maintaining consistent heading capitalization.
|
/gemini review |
There was a problem hiding this comment.
Code Review
This pull request introduces several new documentation files (skills) for the Firebase C++ SDK, covering build instructions, commenting standards, code formatting, and test runner workflows. A correction was suggested for the code formatting documentation to use the proper absl-py boolean flag syntax for disabling file formatting.
Description
Add in some generic skills for cpp
Testing
Made sure gemini could use them.
Type of Change
Place an
xthe applicable box:Notes
Release Notessection ofrelease_build_files/readme.md.