Skip to content

add IWYU#49

Closed
winapiadmin wants to merge 10 commits intomainfrom
winapiadmin-patch-1
Closed

add IWYU#49
winapiadmin wants to merge 10 commits intomainfrom
winapiadmin-patch-1

Conversation

@winapiadmin
Copy link
Copy Markdown
Owner

no functional changes

Added a job to determine if functional changes exist based on modified .cpp or .c files. The build job now depends on this check to skip full tests if no relevant changes are detected.
Add check for 'no functional change' in PR body to skip full tests.
@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

winapiadmin and others added 4 commits March 31, 2026 22:30
* Update position.h

* Replace sstream with initializer_list include

* Apply clang-format

* Remove unused includes from attacks.h

Removed unnecessary includes for bitboard and forward declarations.

* Add necessary includes to attacks.cpp

Added includes for cstddef, initializer_list, utility, and bitboard.

* Apply clang-format

* Update includes in moves_io.h

Replaced includes and added types.h for better type handling.

* Apply clang-format

* Update moves_io.cpp

* Apply clang-format

* Add additional includes to position.h

* Apply clang-format

* Update printers.h

* Update printers.cpp

* Apply clang-format

* Include stdexcept for exception handling

Added include for stdexcept to handle exceptions.

* Apply clang-format

---------

Co-authored-by: GitHub Actions <actions@github.com>
@winapiadmin
Copy link
Copy Markdown
Owner Author

IWYU is currently buggy and not very efficient for some reasons.

@winapiadmin winapiadmin deleted the winapiadmin-patch-1 branch March 31, 2026 16:00
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.

1 participant