Commit f0aa86d7 authored by Yorick Peterse's avatar Yorick Peterse

Remove static analysis linting files

parent a7bc64f4
...@@ -10,5 +10,4 @@ lint-yaml: ...@@ -10,5 +10,4 @@ lint-yaml:
variables: variables:
LINT_PATHS: .gitlab-ci.yml .gitlab/ci lib/gitlab/ci/templates changelogs LINT_PATHS: .gitlab-ci.yml .gitlab/ci lib/gitlab/ci/templates changelogs
script: script:
- '[[ ! -d "ee/" ]] || export LINT_PATHS="$LINT_PATHS ee/changelogs"'
- yamllint -f colored $LINT_PATHS - yamllint -f colored $LINT_PATHS
#!/bin/sh
lint_paths="changelogs/unreleased"
[ -d "ee/" ] && lint_paths="$lint_paths ee/changelogs/unreleased"
invalid_files=$(find $lint_paths -type f -not -name "*.yml" -not -name ".gitkeep")
if [ -n "$invalid_files" ]; then
echo "Changelog files must end in .yml, but these did not:"
echo "$invalid_files" | sed -e "s/^/* /"
exit 1
fi
...@@ -38,8 +38,7 @@ class StaticAnalysis ...@@ -38,8 +38,7 @@ class StaticAnalysis
%w[yarn run block-dependencies] => 0.35, %w[yarn run block-dependencies] => 0.35,
%w[scripts/lint-rugged] => 0.23, %w[scripts/lint-rugged] => 0.23,
%w[scripts/gemfile_lock_changed.sh] => 0.02, %w[scripts/gemfile_lock_changed.sh] => 0.02,
%w[scripts/frontend/check_no_partial_karma_jest.sh] => 0.01, %w[scripts/frontend/check_no_partial_karma_jest.sh] => 0.01
%w[scripts/lint-changelog-filenames] => 0.01
}.reject { |k| k.nil? }.sort_by { |a| -a[1] }.to_h.keys.freeze }.reject { |k| k.nil? }.sort_by { |a| -a[1] }.to_h.keys.freeze
def run_tasks! def run_tasks!
......
...@@ -162,8 +162,6 @@ RSpec.describe Tooling::Danger::ProjectHelper do ...@@ -162,8 +162,6 @@ RSpec.describe Tooling::Danger::ProjectHelper do
'workhorse/main.go' | [:workhorse] 'workhorse/main.go' | [:workhorse]
'workhorse/internal/upload/upload.go' | [:workhorse] 'workhorse/internal/upload/upload.go' | [:workhorse]
'changelogs/foo' | [:none]
'ee/changelogs/foo' | [:none]
'locale/gitlab.pot' | [:none] 'locale/gitlab.pot' | [:none]
'FOO' | [:unknown] 'FOO' | [:unknown]
......
...@@ -38,8 +38,6 @@ module Tooling ...@@ -38,8 +38,6 @@ module Tooling
%r{\A(ee/)?config/feature_flags/} => :feature_flag, %r{\A(ee/)?config/feature_flags/} => :feature_flag,
%r{\A(ee/)?(changelogs/unreleased)(-ee)?/} => :changelog,
%r{\Adoc/development/usage_ping/dictionary\.md\z} => [:docs, :product_intelligence], %r{\Adoc/development/usage_ping/dictionary\.md\z} => [:docs, :product_intelligence],
%r{\Adoc/.*(\.(md|png|gif|jpg))\z} => :docs, %r{\Adoc/.*(\.(md|png|gif|jpg))\z} => :docs,
%r{\A(CONTRIBUTING|LICENSE|MAINTENANCE|PHILOSOPHY|PROCESS|README)(\.md)?\z} => :docs, %r{\A(CONTRIBUTING|LICENSE|MAINTENANCE|PHILOSOPHY|PROCESS|README)(\.md)?\z} => :docs,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment