Merge branch 'master' into mpl-keep-clustering

Author: AcKoucher

.bazelrc

@@ -14,6 +14,7 @@ common --incompatible_disallow_empty_glob
 
 build --cxxopt "-std=c++17"        --host_cxxopt "-std=c++17"
 build --cxxopt "-xc++"             --host_cxxopt "-xc++"
+build --cxxopt "-DBAZEL_BUILD"     --host_cxxopt "-DBAZEL_BUILD"
 
 # Needed for floating point stability in FFT (fft_test will check this).
 # See also https://kristerw.github.io/2021/11/09/fp-contract/
@@ -78,23 +79,18 @@ build --jobs=200
 build --disk_cache=~/.cache/bazel-disk-cache
 build --repository_cache=~/.cache/bazel-repository-cache
 
-# --- CI Settings (write access) ---
-build:ci --remote_cache=https://storage.googleapis.com/openroad-bazel-cache
-build:ci --remote_cache_compression=true
-build:ci --remote_upload_local_results=true
 # CI only needs to know if the build succeeded, not the artifacts themselves.
 build:ci --remote_download_minimal
+build:ci --remote_upload_local_results=true
 # Disable disk cache for CI builds
 build:ci --disk_cache=
 
-# --- OpenROAD Developer Settings (read-only access) ---
-build:openroad-dev --remote_cache=https://storage.googleapis.com/openroad-bazel-cache
-build:openroad-dev --remote_cache_compression=true
-build:openroad-dev --remote_upload_local_results=false
-# Developers need the actual output files to run and use them.
-build:openroad-dev --remote_download_toplevel
-build:openroad-dev --google_default_credentials=true
-build:openroad-dev --disk_cache=/workspace/.cache/bazel-disk-cache
-build:openroad-dev --repository_cache=/workspace/.cache/bazel-repository-cache
+# Setup remote cache
+build --remote_cache=https://bazel.precisioninno.com
+build --remote_cache_compression=true
+build --remote_upload_local_results=false
+
+# Without this, bazelisk build ... builds the bazel-orfs tests
+build --build_tag_filters=-orfs
 
 try-import %workspace%/user.bazelrc

.buildifier.json

@@ -0,0 +1,6 @@
+{
+    "type": "auto",
+    "mode": "fix",
+    "lint": "fix",
+    "warnings": "all"
+}

.clang-format

@@ -27,6 +27,7 @@ BraceWrapping:
 BreakBeforeBinaryOperators: All
 BreakBeforeBraces: Custom
 InsertBraces: true
+LineEnding: LF
 DerivePointerAlignment: false
 SpaceAfterCStyleCast: true
 Standard: c++17

.clang-tidy

@@ -65,7 +65,7 @@ CheckOptions:
 
 # All modules but sta
 # Exclude build as there is too much noise from swig generated code
-HeaderFilterRegex: "(?!build/.*)/(ant|cts|dbSta|dft|dpl|drt|dst|exa|fin|gpl|grt|gui|ifp|mpl|odb|ord|pad|par|pdn|ppl|psm|rcx|rmp|rsz|stt|tap|upf|utl)/.*"
+HeaderFilterRegex: "(?!build/.*)/(ant|cgt|cts|cut|dbSta|dft|dpl|drt|dst|est|exa|fin|gpl|grt|gui|ifp|mpl|odb|ord|pad|par|pdn|ppl|psm|rcx|ram|rmp|rsz|stt|tap|upf|utl)/.*"
 
 # Not currently handling identifier naming
 # WarningsAsErrors: "*"

.github/dependabot.yml

@@ -0,0 +1,10 @@
+version: 2
+updates:
+  - package-ecosystem: "github-actions"
+    directory: "/"
+    schedule:
+      interval: "daily"
+  - package-ecosystem: "gitsubmodule"
+    directory: "/"
+    schedule:
+      interval: "daily"

.github/workflows/black.yaml

@@ -13,6 +13,6 @@ jobs:
     runs-on: ${{ vars.USE_SELF_HOSTED == 'true' && 'self-hosted' || 'ubuntu-latest' }}
     steps:
       - name: Checkout Code
-        uses: actions/checkout@v4
+        uses: actions/checkout@v5
       - name: Run black
         uses: psf/black@stable

.github/workflows/buildifier.yaml

@@ -0,0 +1,44 @@
+name: Lint Bazel
+
+on:
+  push:
+    branches:
+      - master
+  pull_request:
+    branches:
+      - master
+
+jobs:
+  buildifier:
+    runs-on: ${{ vars.USE_SELF_HOSTED == 'true' && 'self-hosted' || 'ubuntu-latest' }}
+    strategy:
+      fail-fast: false
+      matrix:
+        check:
+          - name: Buildifier format
+            run: ./buildifier -r -mode=check -lint=off .
+          - name: Buildifier lint
+            run: ./buildifier -r -lint=warn .
+    name: ${{ matrix.check.name }}
+    env:
+      BUILDIFIER_VERSION: v8.2.1
+
+    steps:
+      - name: Checkout Code
+        uses: actions/checkout@v5
+
+      - name: Cache buildifier
+        id: cache-buildifier
+        uses: actions/cache@v4
+        with:
+          path: ./buildifier
+          key: ${{ runner.os }}-buildifier-${{ env.BUILDIFIER_VERSION }}
+
+      - name: Download buildifier
+        if: steps.cache-buildifier.outputs.cache-hit != 'true'
+        run: |
+          wget https://github.com/bazelbuild/buildtools/releases/download/${BUILDIFIER_VERSION}/buildifier-linux-amd64 -O buildifier
+          chmod +x buildifier
+
+      - name: ${{ matrix.check.name }}
+        run: ${{ matrix.check.run }}

.github/workflows/github-actions-are-differences-found.yml

@@ -8,7 +8,7 @@ jobs:
     runs-on: ${{ vars.USE_SELF_HOSTED == 'true' && 'self-hosted' || 'ubuntu-latest' }}
     steps:
       - name: Check out repository code
-        uses: actions/checkout@v4
+        uses: actions/checkout@v5
         with:
           fetch-depth: 0
       - name: Check ok files

.github/workflows/github-actions-are-odb-files-generated.yml

@@ -13,9 +13,10 @@ jobs:
     runs-on: ${{ vars.USE_SELF_HOSTED == 'true' && 'self-hosted' || 'ubuntu-latest' }}
     steps:
       - name: Check out repository code
-        uses: actions/checkout@v4
+        uses: actions/checkout@v5
         with:
           fetch-depth: 0
+          submodules: 'recursive'
       - name: setup
         run: |
           python3 -m venv venv

.github/workflows/github-actions-clang-tidy-post.yml

@@ -13,7 +13,7 @@ jobs:
     steps:
       # Downloads the artifact uploaded by the lint action
       - name: 'Download artifact'
-        uses: actions/github-script@v6
+        uses: actions/github-script@v8
         with:
           script: |
             const artifacts = await github.rest.actions.listWorkflowRunArtifacts({