Merge pull request #77 from DVLab-NTU/develop

🔖 v0.7.0

.clang-format

@@ -13,6 +13,7 @@ AccessModifierOffset: -4
 
 PackConstructorInitializers: BinPack
 # QualifierAlignment: Right # disabled because it breaks the code in some cases
+DerivePointerAlignment: false
 PointerAlignment: Left
 ReferenceAlignment: Left
 AlignConsecutiveAssignments: Consecutive

.clang-tidy

@@ -6,7 +6,9 @@ Checks: >
   -cppcoreguidelines-avoid-magic-numbers,
   -cppcoreguidelines-owning-memory,
   -cppcoreguidelines-non-private-member-variables-in-classes,
-  performance-noexcept-*,
+  bugprone-assert-side-effect,
+  performance-*,
+  performance-move-const-arg,
   modernize-use-auto,
   misc-const-correctness,
   misc-non-copyable-objects,
@@ -17,12 +19,21 @@ Checks: >
   misc-uniqueptr-reset-release,
   misc-unused-*,
   misc-use-anonymous-namespace,
-  bugprone-assert-side-effect,
-  performance-*
-  readability-*
+  readability-*,
+  -readability-implicit-bool-conversion,
+  -readability-redundant-string-init,
+  -readability-qualified-auto,
+  -readability-function-cognitive-complexity,
+  -readability-else-after-return,
+  -readability-convert-member-functions-to-static,
+  -readability-uppercase-literal-suffix,
+  -readability-isolate-declaration,
+  -readability-braces-around-statements,
+  -readability-magic-numbers,
+  -readability-identifier-length,
+  -readability-avoid-nested-conditional-operator,
 WarningsAsErrors: ""
 HeaderFilterRegex: ".*"
-AnalyzeTemporaryDtors: false
 FormatStyle: none
 CheckOptions:
   - { key: readability-identifier-naming.VariableCase, value: lower_case }

.dockerignore

@@ -0,0 +1,2 @@
+build/
+qsyn

.github/workflows/clang-test-macos.yml

@@ -13,12 +13,12 @@ jobs:
     steps:
       - name: Check out Git repository
         uses: actions/checkout@v3
-      - name: Download llvm-clang
+      - name: Download llvm-clang, parallel and coreutils
         run: |
-          brew install llvm
-          echo 'export PATH="/usr/local/opt/llvm/bin:$PATH"' >> /Users/runner/.bash_profile
-          echo 'export LDFLAGS="-L/usr/local/opt/llvm/lib"' >> /Users/runner/.bash_profile
-          echo 'export CPPFLAGS="-I/usr/local/opt/llvm/include"' >> /Users/runner/.bash_profile
+          brew install llvm parallel coreutils
+          echo 'export PATH="$(brew --prefix)/opt/llvm/bin:$PATH"' >> /Users/runner/.bash_profile
+          echo 'export LDFLAGS="-L$(brew --prefix)/opt/llvm/lib"' >> /Users/runner/.bash_profile
+          echo 'export CPPFLAGS="-I$(brew --prefix)/opt/llvm/include"' >> /Users/runner/.bash_profile
 
       - name: Build qsyn
         run: |
@@ -28,7 +28,5 @@ jobs:
           mkdir -p build
           cmake -S . -B build -DCMAKE_CXX_COMPILER=$(which clang++)
           cmake --build build -j $(sysctl -n hw.logicalcpu)
-      ## remove this comment to run tests
-      ## currently, tests are not working on macos because of implementation of std::unordered_map (we guess)
-      # - name: Run test for qsyn
-      #   run: make test
+      - name: Run test for qsyn
+        run: make test

.gitignore

@@ -5,6 +5,9 @@ qsyn*
 !src/qsyn*
 !src/qsyn/qsyn*
 
+# for work-in-progress things
+wip/
+
 # vscode configuration files and folders
 .vscode