Skip to content

Commit

Permalink
More changes to run on gannett runners and chnage formatting chechks
Browse files Browse the repository at this point in the history
  • Loading branch information
Konstantin Antselovich committed Dec 11, 2024
1 parent 90e469a commit 43fe66a
Show file tree
Hide file tree
Showing 4 changed files with 19 additions and 16 deletions.
1 change: 0 additions & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ on:
- '.github/workflows/**'
- '!dev/**'
pull_request:
branches: [ master ]

workflow_dispatch:

Expand Down
16 changes: 9 additions & 7 deletions .github/workflows/ci_full.yml
Original file line number Diff line number Diff line change
@@ -1,13 +1,12 @@
name: CI-COVER-DATABASES

on:
# push:
# paths:
# - '**.py'
# - '.github/workflows/**'
# - '!dev/**'
push:
paths:
- '**.py'
- '.github/workflows/**'
- '!dev/**'
pull_request:
branches: [ master ]
workflow_dispatch:

permissions:
Expand All @@ -19,7 +18,7 @@ jobs:
strategy:
fail-fast: false
matrix:
os: [ubuntu-latest]
os: [self-hosted, tools-low, cloud-engineering]
python-version:
- "3.11"

Expand All @@ -33,6 +32,9 @@ jobs:
with:
python-version: ${{ matrix.python-version }}

- name: Install Packages
run: chmod +x tests/install-packages.sh && ./tests/install-packages.sh

- name: Build the stack
run: docker-compose up -d mysql postgres presto trino clickhouse vertica

Expand Down
14 changes: 8 additions & 6 deletions .github/workflows/formatter.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ on:
jobs:
linter_name:
name: runner / ruff
runs-on: ubuntu-latest
runs-on: [self-hosted, tools-low, cloud-engineering]
steps:
- name: Checkout PR head
uses: actions/checkout@v3
Expand All @@ -27,6 +27,7 @@ jobs:
uses: chartboost/ruff-action@v1
id: ruff_formatter_suggestions
with:
changed-files: 'true'
args: format --diff

# This only runs if the PR is NOT from a forked repo
Expand All @@ -35,11 +36,12 @@ jobs:
uses: chartboost/ruff-action@v1
id: ruff_formatter
with:
changed-files: 'true'
args: format

# This only runs if the PR is NOT from a forked repo
- name: Auto commit ruff formatting
if: ${{ github.event.pull_request.head.repo.fork == false }}
uses: stefanzweifel/git-auto-commit-action@v5
with:
commit_message: 'style fixes by ruff'
#- name: Auto commit ruff formatting
# if: ${{ github.event.pull_request.head.repo.fork == false }}
# uses: stefanzweifel/git-auto-commit-action@v5
# with:
# commit_message: 'style fixes by ruff'
4 changes: 2 additions & 2 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,10 @@
name = "data-diff"
version = "0.11.2"
description = "Command-line tool and Python library to efficiently diff rows across two different databases."
authors = ["Datafold <[email protected]>"]
authors = ["Datafold <[email protected]>", "LOCALiQ Data Team <[email protected]>"]
license = "MIT"
readme = "README.md"
repository = "https://github.com/datafold/data-diff"
repository = "https://github.com/GannettDigital/data-diff"
classifiers = [
"Intended Audience :: Developers",
"Intended Audience :: Information Technology",
Expand Down

0 comments on commit 43fe66a

Please sign in to comment.