Skip to content

Commit

Permalink
Stop linting toml (#88)
Browse files Browse the repository at this point in the history
We don't use any TOML in this project, so let's disable the linter and
stop installing related tools.

This likely just came in from a copy/pasta of our other pre-commit
configs when setting this up.

Related to https://linear.app/prefect/issue/PLA-289/cycle-2-catch-all
  • Loading branch information
mitchnielsen authored Sep 20, 2024
1 parent 14a031f commit 1a4fa55
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 6 deletions.
1 change: 0 additions & 1 deletion .github/workflows/static-analysis.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@ jobs:

- name: Install tool dependencies (non-mise)
run: |
go install github.com/BurntSushi/toml/cmd/tomlv@latest
go install golang.org/x/tools/cmd/goimports@latest
- name: Build helm dependencies
Expand Down
4 changes: 0 additions & 4 deletions .mise.toml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,3 @@ kustomize = '5.3.0'
pre-commit = '3.8.0'
setup-envtest = '0.17.0'
yamllint = '1.35.1'

# tomlv does not currently have an entry in the registry.
# For now, install with `go`:
# go install github.com/BurntSushi/toml/cmd/tomlv@latest
1 change: 0 additions & 1 deletion .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,6 @@ repos:
# See https://github.com/dnephin/pre-commit-golang/issues/30
# - id: go-vet
- id: go-imports
- id: validate-toml
- id: golangci-lint
# We already run unit tests in tests.yaml, which ensures
# they're called properly by using the `make` target.
Expand Down

0 comments on commit 1a4fa55

Please sign in to comment.