Skip to content

Commit a3d7b62

Browse files
authored
Remove the no cheat CI check (#362)
It fails every time, e.g. https://github.com/databrickslabs/lsql/actions/runs/13569680322/job/37931216993. Can be re-added later. In ucx we use [Python to verify we are not cheating pylint](https://github.com/databrickslabs/ucx/blob/ef45443480a0346efb99b67c2a99199ee4ffe1a4/tests/unit/no_cheat.py#L7)
1 parent 2410b4a commit a3d7b62

File tree

1 file changed

+0
-20
lines changed

1 file changed

+0
-20
lines changed

.github/workflows/push.yml

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -60,23 +60,3 @@ jobs:
6060
6161
- name: Fail on differences
6262
run: git diff --exit-code
63-
64-
no-lint-disabled:
65-
runs-on: ubuntu-latest
66-
steps:
67-
- name: Checkout
68-
uses: actions/checkout@v4.2.2
69-
with:
70-
fetch-depth: 0
71-
72-
- name: Verify no lint disabled in the new code
73-
run: |
74-
NEW_CODE=$(git diff origin/main..$(git branch --show-current) | grep -e '^+')
75-
echo "Code diff:"
76-
echo "${NEW_CODE}"
77-
CHEAT=$(echo "${NEW_CODE}" | grep '# pylint: disable' | grep -v "CHEAT" | wc -c)
78-
echo "# cheats: ${CHEAT}"
79-
if [ "${CHEAT}" -ne 0 ]; then
80-
echo "Do not cheat the linter: ${CHEAT}"
81-
exit 1
82-
fi

0 commit comments

Comments
 (0)