Merge pull request #50 from kumvijaya/current

Test 1

2 files changed, 26 insertions, 58 deletions

diff --git a/.github/workflows/ruff-lint-2.yml b/.github/workflows/ruff-lint-2.yml
deleted file mode 100644
index 1c5d54e..0000000
--- a/.github/workflows/ruff-lint-2.yml
+++ /dev/null
@@ -1,56 +0,0 @@
----
-name: Ruff Lint 2
-
-on:
-  pull_request:
-    branches:
-      - current
-
-permissions:
-  pull-requests: write
-  contents: read
-
-jobs:
-  ruff-lint-2:
-    runs-on: ubuntu-latest
-    permissions:
-      pull-requests: write
-      contents: read
-    steps:
-      - name: Checkout reusable actions repo
-        uses: actions/checkout@v3
-        with:
-          repository: vyos/.github
-          path: reusable-actions
-          ref: current
-
-      - name: Checkout head
-        uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
-          fetch-tags: true
-          ref: ${{ github.event.pull_request.head.ref }}
-          repository: ${{ github.event.pull_request.head.repo.full_name }}
-          path: repo
-
-      - name: Fetch base
-        run: |
-          cd repo
-          git fetch https://github.com/${{ github.event.pull_request.base.repo.full_name }} ${{ github.event.pull_request.base.ref }}:refs/remotes/origin/base
-
-      - name: Setup python
-        uses: actions/setup-python@v5
-        with:
-          python-version: '3.11' 
-
-      - name: Get changed files
-        id: changed-files
-        run: |
-          cd repo
-          git diff --name-only origin/base...${{ github.event.pull_request.head.ref }} > changed_files.txt
-
-      - name: Run ruff on changed files
-        uses: ChartBoost/ruff-action@v1
-        with:
-          files: |
-            $(cat repo/changed_files.txt)
\ No newline at end of file
diff --git a/.github/workflows/ruff-lint.yml b/.github/workflows/ruff-lint.yml
index 4a87521..6ba9447 100644
--- a/.github/workflows/ruff-lint.yml
+++ b/.github/workflows/ruff-lint.yml
@@ -6,10 +6,34 @@ on: pull_request
 jobs:
   ruff-lint:
     runs-on: ubuntu-latest
+    permissions:
+      pull-requests: write
+      contents: read
     steps:
-      - uses: actions/checkout@v4
+      - name: Checkout Head
+        uses: actions/checkout@v4
         with:
           fetch-depth: 0
-      - uses: chartboost/ruff-action@491342200cdd1cf4d5132a30ddc546b3b5bc531b
+          ref: ${{ github.event.pull_request.head.ref }}
+          repository: ${{ github.event.pull_request.head.repo.full_name }}
+
+      - name: Get changed py files
+        id: changed-py-files
+        uses: tj-actions/changed-files@v44
+        with:
+          files: |
+             **.py
+
+      - name: Ruff Check
+        if: steps.changed-py-files.outputs.any_changed == 'true'
+        uses: chartboost/ruff-action@491342200cdd1cf4d5132a30ddc546b3b5bc531b
         with:
+          args: check
           changed-files: 'true'
+
+      - name: Ruff Format
+        if: always() && steps.changed-py-files.outputs.any_changed == 'true'
+        uses: chartboost/ruff-action@491342200cdd1cf4d5132a30ddc546b3b5bc531b
+        with:
+          args: format --diff
+          changed-files: 'true'
\ No newline at end of file