pack_stats 0.1.1 → 0.1.3
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
checksums.yaml
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
---
|
2
2
|
SHA256:
|
3
|
-
metadata.gz:
|
4
|
-
data.tar.gz:
|
3
|
+
metadata.gz: f060837410e7daee8e07d54e9bbee788401954480eeb0f9f3b2a8222e6216ae6
|
4
|
+
data.tar.gz: a72d940a47cb17e98efcb0d658799853a3f295674326caf2f6eca22ee23bacec
|
5
5
|
SHA512:
|
6
|
-
metadata.gz:
|
7
|
-
data.tar.gz:
|
6
|
+
metadata.gz: ebc898c2c55afc4ebb1dc19c7f538cde5667d5bc8e64b7d06163a5d04baf9f6dc867e6038a7e9fa19bc6785b935e86e05499bebe364c5586fec1d56febbb6d53
|
7
|
+
data.tar.gz: c16aec135614d559e053410961a74bcbc1ad9d7a25ac02173b72153235a8639fc1a898f0579239840dca39084ad0c35a924075b99da64c2e6aa75ffca986bf49
|
@@ -6,7 +6,6 @@ require 'pack_stats/private/metrics'
|
|
6
6
|
require 'pack_stats/private/metrics/files'
|
7
7
|
require 'pack_stats/private/metrics/public_usage'
|
8
8
|
require 'pack_stats/private/metrics/packwerk_checker_usage'
|
9
|
-
require 'pack_stats/private/metrics/rubocop_usage'
|
10
9
|
require 'pack_stats/private/metrics/dependencies'
|
11
10
|
require 'pack_stats/private/metrics/packages'
|
12
11
|
require 'pack_stats/private/metrics/packages_by_team'
|
@@ -31,7 +31,6 @@ module PackStats
|
|
31
31
|
all_metrics << GaugeMetric.for('all_packages.has_readme.count', packages.count { |package| Metrics.has_readme?(package) }, package_tags)
|
32
32
|
|
33
33
|
all_metrics += Metrics::PackwerkCheckerUsage.get_checker_metrics('all_packages', packages, package_tags)
|
34
|
-
all_metrics += Metrics::RubocopUsage.get_metrics('all_packages', packages, package_tags)
|
35
34
|
all_metrics << GaugeMetric.for('all_packages.package_based_file_ownership.count', packages.count { |package| !package.metadata['owner'].nil? }, package_tags)
|
36
35
|
|
37
36
|
inbound_violations_by_package = packages.flat_map(&:violations).group_by(&:to_package_name)
|
@@ -59,7 +58,10 @@ module PackStats
|
|
59
58
|
checker.violation_type_tag
|
60
59
|
]
|
61
60
|
|
62
|
-
|
61
|
+
count = Metrics.file_count(violations)
|
62
|
+
if count > 0
|
63
|
+
all_metrics << GaugeMetric.for("by_package.violations.by_other_package.count", Metrics.file_count(violations), tags)
|
64
|
+
end
|
63
65
|
end
|
64
66
|
when PackwerkCheckerUsage::Direction::Inbound
|
65
67
|
all_violations_of_type = inbound_violations.select { |v| v.type == checker.violation_type }
|
@@ -72,7 +74,10 @@ module PackStats
|
|
72
74
|
checker.violation_type_tag
|
73
75
|
]
|
74
76
|
|
75
|
-
|
77
|
+
count = Metrics.file_count(violations)
|
78
|
+
if count > 0
|
79
|
+
all_metrics << GaugeMetric.for("by_package.violations.by_other_package.count", count, tags)
|
80
|
+
end
|
76
81
|
end
|
77
82
|
else
|
78
83
|
T.absurd(direction)
|
@@ -44,7 +44,10 @@ module PackStats
|
|
44
44
|
all_packages.group_by { |package| Private.package_owner(package) }.each do |other_team_name, other_teams_packages|
|
45
45
|
violations = outbound_violations.select{|v| other_teams_packages.map(&:name).include?(v.to_package_name) && v.type == checker.violation_type}
|
46
46
|
tags = team_tags + Metrics.tags_for_other_team(other_team_name) + [checker.violation_type_tag]
|
47
|
-
|
47
|
+
count = Metrics.file_count(violations)
|
48
|
+
if count > 0
|
49
|
+
all_metrics << GaugeMetric.for("by_team.violations.by_other_team.count", count, tags)
|
50
|
+
end
|
48
51
|
end
|
49
52
|
when PackwerkCheckerUsage::Direction::Inbound
|
50
53
|
all_violations_of_type = inbound_violations.select { |v| v.type == checker.violation_type }
|
@@ -56,7 +59,10 @@ module PackStats
|
|
56
59
|
all_packages.group_by { |package| Private.package_owner(package) }.each do |other_team_name, other_teams_packages|
|
57
60
|
violations = other_teams_packages.flat_map(&:violations).select{|v| packages_for_team.map(&:name).include?(v.to_package_name) && v.type == checker.violation_type}
|
58
61
|
tags = team_tags + Metrics.tags_for_other_team(other_team_name) + [checker.violation_type_tag]
|
59
|
-
|
62
|
+
count = Metrics.file_count(violations)
|
63
|
+
if count > 0
|
64
|
+
all_metrics << GaugeMetric.for("by_team.violations.by_other_team.count", count, tags)
|
65
|
+
end
|
60
66
|
end
|
61
67
|
else
|
62
68
|
T.absurd(direction)
|
metadata
CHANGED
@@ -1,14 +1,14 @@
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
2
2
|
name: pack_stats
|
3
3
|
version: !ruby/object:Gem::Version
|
4
|
-
version: 0.1.
|
4
|
+
version: 0.1.3
|
5
5
|
platform: ruby
|
6
6
|
authors:
|
7
7
|
- Gusto Engineers
|
8
8
|
autorequire:
|
9
9
|
bindir: bin
|
10
10
|
cert_chain: []
|
11
|
-
date: 2023-
|
11
|
+
date: 2023-07-26 00:00:00.000000000 Z
|
12
12
|
dependencies:
|
13
13
|
- !ruby/object:Gem::Dependency
|
14
14
|
name: code_teams
|
@@ -198,7 +198,6 @@ files:
|
|
198
198
|
- lib/pack_stats/private/metrics/packages_by_team.rb
|
199
199
|
- lib/pack_stats/private/metrics/packwerk_checker_usage.rb
|
200
200
|
- lib/pack_stats/private/metrics/public_usage.rb
|
201
|
-
- lib/pack_stats/private/metrics/rubocop_usage.rb
|
202
201
|
- lib/pack_stats/private/source_code_file.rb
|
203
202
|
- lib/pack_stats/tag.rb
|
204
203
|
- lib/pack_stats/tags.rb
|
@@ -1,98 +0,0 @@
|
|
1
|
-
# typed: strict
|
2
|
-
# frozen_string_literal: true
|
3
|
-
|
4
|
-
module PackStats
|
5
|
-
module Private
|
6
|
-
module Metrics
|
7
|
-
class RubocopUsage
|
8
|
-
extend T::Sig
|
9
|
-
|
10
|
-
sig { params(prefix: String, packages: T::Array[ParsePackwerk::Package], package_tags: T::Array[Tag]).returns(T::Array[GaugeMetric]) }
|
11
|
-
def self.get_metrics(prefix, packages, package_tags)
|
12
|
-
[
|
13
|
-
*get_rubocop_exclusions(prefix, packages, package_tags),
|
14
|
-
*get_rubocop_usage_metrics(prefix, packages, package_tags)
|
15
|
-
]
|
16
|
-
end
|
17
|
-
|
18
|
-
sig { params(prefix: String, packages: T::Array[ParsePackwerk::Package], package_tags: T::Array[Tag]).returns(T::Array[GaugeMetric]) }
|
19
|
-
def self.get_rubocop_usage_metrics(prefix, packages, package_tags)
|
20
|
-
metrics = T.let([], T::Array[GaugeMetric])
|
21
|
-
|
22
|
-
rubocops.each do |cop_name|
|
23
|
-
['false', 'true', 'strict'].each do |enabled_mode|
|
24
|
-
count_of_packages = ParsePackwerk.all.count do |package|
|
25
|
-
# We will likely want a rubocop-packs API for this, to be able to ask if a cop is enabled for a pack.
|
26
|
-
# It's possible we will want to allow these to be enabled at the top-level `.rubocop.yml`,
|
27
|
-
# in which case we wouldn't get the right metrics with this approach. However, we can also accept
|
28
|
-
# that as a current limitation.
|
29
|
-
rubocop_yml_file = package.directory.join(RuboCop::Packs::PACK_LEVEL_RUBOCOP_YML)
|
30
|
-
next false if !rubocop_yml_file.exist?
|
31
|
-
rubocop_yml = YAML.load_file(rubocop_yml_file)
|
32
|
-
cop_config = rubocop_yml[cop_name]
|
33
|
-
|
34
|
-
strict_mode = cop_config && cop_config['FailureMode'] == 'strict'
|
35
|
-
enabled = cop_config && cop_config['Enabled']
|
36
|
-
case enabled_mode
|
37
|
-
when 'false'
|
38
|
-
!enabled
|
39
|
-
when 'true'
|
40
|
-
enabled && !strict_mode
|
41
|
-
when 'strict'
|
42
|
-
!!strict_mode
|
43
|
-
end
|
44
|
-
end
|
45
|
-
|
46
|
-
metric_name = "#{prefix}.rubocops.#{to_tag_name(cop_name)}.#{enabled_mode}.count"
|
47
|
-
metrics << GaugeMetric.for(metric_name, count_of_packages, package_tags)
|
48
|
-
end
|
49
|
-
end
|
50
|
-
|
51
|
-
metrics
|
52
|
-
end
|
53
|
-
|
54
|
-
sig { params(prefix: String, packages: T::Array[ParsePackwerk::Package], package_tags: T::Array[Tag]).returns(T::Array[GaugeMetric]) }
|
55
|
-
def self.get_rubocop_exclusions(prefix, packages, package_tags)
|
56
|
-
rubocops.flat_map do |cop_name|
|
57
|
-
metric_name = "#{prefix}.rubocops.#{to_tag_name(cop_name)}.exclusions.count"
|
58
|
-
all_exclusions_count = ParsePackwerk.all.sum { |package| exclude_count_for_package_and_protection(package, cop_name)}
|
59
|
-
GaugeMetric.for(metric_name, all_exclusions_count, package_tags)
|
60
|
-
end
|
61
|
-
end
|
62
|
-
|
63
|
-
# TODO: `rubocop-packs` may want to expose API for this
|
64
|
-
sig { params(package: ParsePackwerk::Package, cop_name: String).returns(Integer) }
|
65
|
-
def self.exclude_count_for_package_and_protection(package, cop_name)
|
66
|
-
if package.name == ParsePackwerk::ROOT_PACKAGE_NAME
|
67
|
-
rubocop_todo = package.directory.join('.rubocop_todo.yml')
|
68
|
-
else
|
69
|
-
rubocop_todo = package.directory.join(RuboCop::Packs::PACK_LEVEL_RUBOCOP_TODO_YML)
|
70
|
-
end
|
71
|
-
|
72
|
-
if rubocop_todo.exist?
|
73
|
-
loaded_rubocop_todo = YAML.load_file(rubocop_todo)
|
74
|
-
cop_config = loaded_rubocop_todo.fetch(cop_name, {})
|
75
|
-
cop_config.fetch('Exclude', []).count
|
76
|
-
else
|
77
|
-
0
|
78
|
-
end
|
79
|
-
end
|
80
|
-
|
81
|
-
sig { returns(T::Array[String])}
|
82
|
-
def self.rubocops
|
83
|
-
[
|
84
|
-
'Packs/ClassMethodsAsPublicApis',
|
85
|
-
'Packs/RootNamespaceIsPackName',
|
86
|
-
'Packs/TypedPublicApis',
|
87
|
-
'Packs/DocumentedPublicApis',
|
88
|
-
]
|
89
|
-
end
|
90
|
-
|
91
|
-
sig { params(cop_name: String).returns(String) }
|
92
|
-
def self.to_tag_name(cop_name)
|
93
|
-
cop_name.gsub('/', '_').downcase
|
94
|
-
end
|
95
|
-
end
|
96
|
-
end
|
97
|
-
end
|
98
|
-
end
|