Skip to content

fix(ci): temporarily disable taxonomy check - #31950

Merged
timgl merged 1 commit into
masterfrom
tmp-disable-taxonomy-check
May 6, 2025
Merged

timgl merged 1 commit into
masterfrom
tmp-disable-taxonomy-check

Conversation

@thmsobrmlr

Copy link
Copy Markdown
Collaborator

Problem

https://posthog.slack.com/archives/C0113360FFV/p1746531608269629?thread_ts=1744218127.802669&cid=C0113360FFV

Changes

Temporarily disables the CI check.

How did you test this code?

CI run

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

Temporarily disables the taxonomy file validation check in the Python code quality GitHub Action while maintaining all other code quality checks.

  • Commented out the Check if "taxonomy.json/taxonomy.tsx" is up to date step in /.github/actions/python-code-quality/action.yml
  • This change should be treated as temporary to avoid potential inconsistencies between source and generated taxonomy files
  • Consider adding a tracking issue or timeline for re-enabling this check to maintain code quality standards

1 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings | Greptile

Comment on lines +67 to +70
# - name: Check if "taxonomy.json/taxonomy.tsx" is up to date
# shell: bash
# run: |
# npm run taxonomy:build:json && git diff --exit-code

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

style: Disabling CI checks should include a TODO comment with a timeline or ticket reference for re-enabling

@timgl
timgl merged commit 10d4ca1 into master May 6, 2025
@timgl
timgl deleted the tmp-disable-taxonomy-check branch May 6, 2025 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants