From 86a1875d6dce5800e743d76112c560fee60af6df Mon Sep 17 00:00:00 2001 From: Emi Matchu <emi@matchu.dev> Date: Sun, 12 Jan 2025 19:21:55 -0800 Subject: [PATCH] Tweak gem grouping for test libraries Idk a clear and concise way to group all these together, *and* include `rspec-rails` in both development and test as the install instructions recommend, but omit `webmock` from development. Not that it matters a lot, I just want to be minimal about the gems we include, for the sake of clarity. This formulation is a bit verbose, but I think it's maximally clear and self-consistent, and I appreciate that. I might change my mind later though? Anyway, I mostly just noticed this because of the redundancy where `webmock`'s `group: :test` I think doesn't do anything? Shrug --- Gemfile | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/Gemfile b/Gemfile index 24ba48be..c7450146 100644 --- a/Gemfile +++ b/Gemfile @@ -98,5 +98,7 @@ end # For automated tests. group :development, :test do gem "rspec-rails", "~> 7.0" - gem "webmock", "~> 3.24", group: :test +end +group :test do + gem "webmock", "~> 3.24" end