Skip to content

Commit

Permalink
fixing violations
Browse files Browse the repository at this point in the history
  • Loading branch information
anmarchenko committed Jun 3, 2024
1 parent 50a46d4 commit e83688a
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 17 deletions.
16 changes: 4 additions & 12 deletions Appraisals
Original file line number Diff line number Diff line change
Expand Up @@ -3,27 +3,19 @@ $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)

module DisableBundleCheck
def check_command
["bundle", "exec", "false"]
%w[bundle exec false]
end
end

if ["true", "y", "yes", "1"].include?(ENV["APPRAISAL_SKIP_BUNDLE_CHECK"])
if %w[true y yes 1].include?(ENV["APPRAISAL_SKIP_BUNDLE_CHECK"])
::Appraisal::Appraisal.prepend(DisableBundleCheck)
end

alias original_appraise appraise

REMOVED_GEMS = {
check: [
"rbs",
"steep"
],
development: [
"ruby-lsp",
"ruby-lsp-rspec",
"debug",
"irb"
]
check: %w[rbs steep],
development: %w[ruby-lsp ruby-lsp-rspec debug irb]
}

def appraise(group, &block)
Expand Down
4 changes: 2 additions & 2 deletions lib/datadog/ci/test_visibility/serializers/span.rb
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,11 @@ module CI
module TestVisibility
module Serializers
class Span < Base
CONTENT_FIELDS = (["trace_id", "span_id", "parent_id"] + Base::CONTENT_FIELDS).freeze
CONTENT_FIELDS = (%w[trace_id span_id parent_id] + Base::CONTENT_FIELDS).freeze

CONTENT_MAP_SIZE = calculate_content_map_size(CONTENT_FIELDS)

REQUIRED_FIELDS = (["trace_id", "span_id"] + Base::REQUIRED_FIELDS).freeze
REQUIRED_FIELDS = (%w[trace_id span_id] + Base::REQUIRED_FIELDS).freeze

def content_fields
CONTENT_FIELDS
Expand Down
5 changes: 2 additions & 3 deletions static-analysis.datadog.yml
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
schema-version: v1
rulesets:
- ruby-code-style
- ruby-security
Expand All @@ -8,6 +9,4 @@ rulesets:
- '**'
hash-fetch:
ignore:
- 'spec/**/*'

ignorePaths: []
- spec/**/*

0 comments on commit e83688a

Please sign in to comment.