Skip to content

Conversation

@coderabbitai
Copy link
Contributor

@coderabbitai coderabbitai bot commented Nov 24, 2025

Docstrings generation was requested by @ystade.

The following files were modified:

  • include/mqt-core/qdmi/sc/Device.hpp
  • src/na/device/Device.cpp
  • src/qdmi/Driver.cpp
  • src/qdmi/sc/App.cpp
  • src/qdmi/sc/Device.cpp
  • src/qdmi/sc/DynDevice.cpp
  • src/qdmi/sc/Generator.cpp
  • test/na/device/test_app.cpp
  • test/na/device/test_device.cpp
  • test/na/device/test_generator.cpp
  • test/qdmi/sc/test_app.cpp
  • test/qdmi/sc/test_device.cpp
  • test/qdmi/sc/test_generator.cpp
  • test/qdmi/test_driver.cpp
These file types are not supported
  • json/sc/device.json
  • src/qdmi/CMakeLists.txt
  • src/qdmi/sc/CMakeLists.txt
  • test/qdmi/CMakeLists.txt
  • test/qdmi/sc/CMakeLists.txt
ℹ️ Note

CodeRabbit cannot perform edits on its own pull requests yet.

@coderabbitai coderabbitai bot requested a review from ystade November 24, 2025 11:59
@coderabbitai coderabbitai bot added this to the QDMI Support milestone Nov 24, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 24, 2025

Cpp-Linter Report ⚠️

Some files did not pass the configured checks!

clang-tidy (v21.1.7) reports: 6 concern(s)
  • src/qdmi/sc/Generator.cpp:17:1: warning: [misc-include-cleaner]

    included header algorithm is not used directly

       17 | #include <algorithm>
          | ^~~~~~~~~~~~~~~~~~~~
       18 | #include <cassert>
  • src/qdmi/sc/Generator.cpp:20:1: warning: [misc-include-cleaner]

    included header cstddef is not used directly

       20 | #include <cstddef>
          | ^~~~~~~~~~~~~~~~~~
       21 | #include <cstdint>
  • src/qdmi/sc/Generator.cpp:23:1: warning: [misc-include-cleaner]

    included header functional is not used directly

       23 | #include <functional>
          | ^~~~~~~~~~~~~~~~~~~~~
       24 | #include <istream>
  • src/qdmi/sc/Generator.cpp:27:1: warning: [misc-include-cleaner]

    included header ranges is not used directly

       27 | #include <ranges>
          | ^~~~~~~~~~~~~~~~~
       28 | #include <spdlog/spdlog.h>
  • src/qdmi/sc/Generator.cpp:32:1: warning: [misc-include-cleaner]

    included header tuple is not used directly

       32 | #include <tuple>
          | ^~~~~~~~~~~~~~~~
       33 | #include <utility>
  • src/qdmi/sc/Generator.cpp:33:1: warning: [misc-include-cleaner]

    included header utility is not used directly

       33 | #include <utility>
          | ^~~~~~~~~~~~~~~~~~
       34 | #include <vector>

Have any feedback or feature suggestions? Share it here.

@ystade ystade force-pushed the coderabbitai/docstrings/8553ccb branch from 6a38a0f to cbf70e9 Compare November 24, 2025 12:14
@ystade ystade merged commit 19e699b into ystade/qdmi-sc-device Nov 24, 2025
1 of 3 checks passed
@ystade ystade deleted the coderabbitai/docstrings/8553ccb branch November 24, 2025 12:14
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