Skip to content

Commit 73381f3

Browse files
authored
Merge pull request #2203 from Shopify/uk-consider-more-subconstants
Stop ignoring symbols in payload when discovering subconstants
2 parents c01487c + 21dd01c commit 73381f3

File tree

72 files changed

+48545
-46393
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

72 files changed

+48545
-46393
lines changed

Gemfile.lock

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -170,7 +170,7 @@ GEM
170170
pp (>= 0.6.0)
171171
rdoc (>= 4.0.0)
172172
reline (>= 0.4.2)
173-
json (2.9.1)
173+
json (2.10.1)
174174
json_api_client (1.23.0)
175175
activemodel (>= 6.0.0)
176176
activesupport (>= 6.0.0)

lib/tapioca/gem/listeners/subconstants.rb

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,11 +14,10 @@ class Subconstants < Base
1414
sig { override.params(event: ScopeNodeAdded).void }
1515
def on_scope(event)
1616
symbol = event.symbol
17-
return if @pipeline.symbol_in_payload?(symbol) && event.node.empty?
17+
constant = event.constant
1818

1919
prefix = symbol == "Object" ? "" : symbol
2020

21-
constant = event.constant
2221
constants_of(constant).sort.uniq.map do |constant_name|
2322
name = "#{prefix}::#{constant_name}"
2423
subconstant = constantize(name)

0 commit comments

Comments
 (0)