Skip to content

Commit ee40e9f

Browse files
committed
Merge pull request #1173 from bf4/comment_private_reader_warnings
Comment private accessor warnings
2 parents 61c54bd + c9ae868 commit ee40e9f

File tree

1 file changed

+21
-0
lines changed

1 file changed

+21
-0
lines changed

lib/active_model_serializers.rb

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,27 @@ module ActiveModelSerializers
99

1010
module_function
1111

12+
# @note
13+
# ```ruby
14+
# private
15+
#
16+
# attr_reader :resource, :adapter_opts, :serializer_opts
17+
# ```
18+
#
19+
# Will generate a warning, though it shouldn't.
20+
# There's a bug in Ruby for this: https://bugs.ruby-lang.org/issues/10967
21+
#
22+
# We can use +ActiveModelSerializers.silence_warnings+ as a
23+
# 'safety valve' for unfixable or not-worth-fixing warnings,
24+
# and keep our app warning-free.
25+
#
26+
# ```ruby
27+
# private
28+
#
29+
# ActiveModelSerializers.silence_warnings do
30+
# attr_reader :resource, :adapter_opts, :serializer_opts
31+
# end
32+
# ```
1233
def silence_warnings
1334
verbose = $VERBOSE
1435
$VERBOSE = nil

0 commit comments

Comments
 (0)