diff --git a/.rubocop.yml b/.rubocop.yml index e67a5022d1d9488804cc8f11653191a2a64df6b4..ad53188f34af40e27cdf8b737ca67843874dd0b0 100644 --- a/.rubocop.yml +++ b/.rubocop.yml @@ -29,7 +29,8 @@ inherit_mode: AllCops: # The oldest supported Ruby version. - TargetRubyVersion: 3.2 + TargetRubyVersion: 3.3 + ParserEngine: parser_prism TargetRailsVersion: 7.0 DefaultFormatter: <%= ENV['CI'] == '1' ? 'clang' : 'progress' %> Exclude: diff --git a/Gemfile b/Gemfile index 3a45a24b241ce3ffa073f6b9f4d23f078055a721..afb1a531fd4656abc9ccf0264868cbaf14e96132 100644 --- a/Gemfile +++ b/Gemfile @@ -519,6 +519,7 @@ group :development, :test do gem 'spring', '~> 4.3.0', feature_category: :shared gem 'spring-commands-rspec', '~> 1.0.4', feature_category: :shared + gem 'prism', '~> 1.4.0', feature_category: :tooling gem 'gitlab-styles', '~> 13.1.0', feature_category: :tooling, require: false gem 'haml_lint', '~> 0.58', feature_category: :tooling, require: false diff --git a/Gemfile.checksum b/Gemfile.checksum index 1a4f45f24db9540eacf7b99cbdf09b0e920e9903..c682b9111d53f43a91fd9b3e47550e97bf9e5ef8 100644 --- a/Gemfile.checksum +++ b/Gemfile.checksum @@ -526,7 +526,7 @@ {"name":"premailer-rails","version":"1.12.0","platform":"ruby","checksum":"c13815d161b9bc7f7d3d81396b0bb0a61a90fa9bd89931548bf4e537c7710400"}, {"name":"prettyprint","version":"0.2.0","platform":"ruby","checksum":"2bc9e15581a94742064a3cc8b0fb9d45aae3d03a1baa6ef80922627a0766f193"}, {"name":"prime","version":"0.1.3","platform":"ruby","checksum":"baf031c50d6ce923594913befc8ac86a3251bffb9d6a5e8b03687962054e53e3"}, -{"name":"prism","version":"1.2.0","platform":"ruby","checksum":"24ff9cd3232346e68052659f14c9a618022ea98935f774df465206aba5c06d2f"}, +{"name":"prism","version":"1.4.0","platform":"ruby","checksum":"dc0e3e00e93160213dc2a65519d9002a4a1e7b962db57d444cf1a71565bb703e"}, {"name":"proc_to_ast","version":"0.1.0","platform":"ruby","checksum":"92a73fa66e2250a83f8589f818b0751bcf227c68f85916202df7af85082f8691"}, {"name":"prometheus-client-mmap","version":"1.2.10","platform":"aarch64-linux-gnu","checksum":"a014e7748ba72928e58538f155b18479ce315396e9c7ab2115a80ea0a86001ac"}, {"name":"prometheus-client-mmap","version":"1.2.10","platform":"aarch64-linux-musl","checksum":"ba6c4b091c8a249b207ed7e482bcf58095b2631d29c3e51a463ab7f6cc01b8bd"}, diff --git a/Gemfile.lock b/Gemfile.lock index b65917a5f973b690667ada2c596ba8ead3f59a99..64c4fba9e5193349cbb5891255b4fc0a70a58852 100644 --- a/Gemfile.lock +++ b/Gemfile.lock @@ -1486,7 +1486,7 @@ GEM prime (0.1.3) forwardable singleton - prism (1.2.0) + prism (1.4.0) proc_to_ast (0.1.0) coderay parser @@ -2317,6 +2317,7 @@ DEPENDENCIES prawn prawn-svg premailer-rails (~> 1.12.0) + prism (~> 1.4.0) prometheus-client-mmap (~> 1.2.8) pry-byebug pry-rails (~> 0.3.9) diff --git a/Gemfile.next.checksum b/Gemfile.next.checksum index 98483921c7f39fbe1dfd098bd4b31cccb55c28a2..a890159834c77e431be2fe56bb86657677dd7394 100644 --- a/Gemfile.next.checksum +++ b/Gemfile.next.checksum @@ -526,7 +526,7 @@ {"name":"premailer-rails","version":"1.12.0","platform":"ruby","checksum":"c13815d161b9bc7f7d3d81396b0bb0a61a90fa9bd89931548bf4e537c7710400"}, {"name":"prettyprint","version":"0.2.0","platform":"ruby","checksum":"2bc9e15581a94742064a3cc8b0fb9d45aae3d03a1baa6ef80922627a0766f193"}, {"name":"prime","version":"0.1.3","platform":"ruby","checksum":"baf031c50d6ce923594913befc8ac86a3251bffb9d6a5e8b03687962054e53e3"}, -{"name":"prism","version":"1.2.0","platform":"ruby","checksum":"24ff9cd3232346e68052659f14c9a618022ea98935f774df465206aba5c06d2f"}, +{"name":"prism","version":"1.4.0","platform":"ruby","checksum":"dc0e3e00e93160213dc2a65519d9002a4a1e7b962db57d444cf1a71565bb703e"}, {"name":"proc_to_ast","version":"0.1.0","platform":"ruby","checksum":"92a73fa66e2250a83f8589f818b0751bcf227c68f85916202df7af85082f8691"}, {"name":"prometheus-client-mmap","version":"1.2.10","platform":"aarch64-linux-gnu","checksum":"a014e7748ba72928e58538f155b18479ce315396e9c7ab2115a80ea0a86001ac"}, {"name":"prometheus-client-mmap","version":"1.2.10","platform":"aarch64-linux-musl","checksum":"ba6c4b091c8a249b207ed7e482bcf58095b2631d29c3e51a463ab7f6cc01b8bd"}, diff --git a/Gemfile.next.lock b/Gemfile.next.lock index 3d1d266f5af951147c88333fdcb8ed0b9b0ce9be..e8add5c7a2e5ea6daa17316804a308fbc23e61db 100644 --- a/Gemfile.next.lock +++ b/Gemfile.next.lock @@ -1480,7 +1480,7 @@ GEM prime (0.1.3) forwardable singleton - prism (1.2.0) + prism (1.4.0) proc_to_ast (0.1.0) coderay parser @@ -2312,6 +2312,7 @@ DEPENDENCIES prawn prawn-svg premailer-rails (~> 1.12.0) + prism (~> 1.4.0) prometheus-client-mmap (~> 1.2.8) pry-byebug pry-rails (~> 0.3.9) diff --git a/ee/app/views/groups/saml_providers/_test_button.html.haml b/ee/app/views/groups/saml_providers/_test_button.html.haml index 978c3117f63642e27fa1de143466236f0658d0d5..ffb30338898db4b9870d72bfca507a7bde72f209 100644 --- a/ee/app/views/groups/saml_providers/_test_button.html.haml +++ b/ee/app/views/groups/saml_providers/_test_button.html.haml @@ -1,4 +1,4 @@ -= return unless saml_provider.persisted? +- return unless saml_provider.persisted? = saml_link_for_provider _('Verify SAML Configuration'), saml_provider, redirect: ::OmniAuth::Strategies::GroupSaml::VERIFY_SAML_RESPONSE, data: { testid: 'saml-settings-test-button' } = render Pajamas::ButtonComponent.new(button_options: { class: 'gl-hidden', disabled: true }) do diff --git a/ee/spec/lib/gitlab/llm/utils/code_suggestion_formatter_spec.rb b/ee/spec/lib/gitlab/llm/utils/code_suggestion_formatter_spec.rb index 568c35f4168b344d2ebb3ab1827bc721ab6985f7..a31accbf8f85e6f881dfbb7a2dcdb3bfdbb633eb 100644 --- a/ee/spec/lib/gitlab/llm/utils/code_suggestion_formatter_spec.rb +++ b/ee/spec/lib/gitlab/llm/utils/code_suggestion_formatter_spec.rb @@ -174,7 +174,7 @@ it 'returns the comment without the suggestions' do expect(parse[:from]).to eq <<~FROM - \ + \ first offending line second offending line third offending line diff --git a/ee/spec/requests/ee/admin/groups_controller_spec.rb b/ee/spec/requests/ee/admin/groups_controller_spec.rb index 90d79bb55736953b651fffebeaf3b4949a62a7f5..d1f826a2106e8245d6e45f5a4e5b6b4618db3324 100644 --- a/ee/spec/requests/ee/admin/groups_controller_spec.rb +++ b/ee/spec/requests/ee/admin/groups_controller_spec.rb @@ -2,7 +2,7 @@ require 'spec_helper' -RSpec.describe Admin::GroupsController, :enable_admin_mode, type: :request, feature_category: :groups_and_projects do +RSpec.describe Admin::GroupsController, :enable_admin_mode, feature_category: :groups_and_projects do let_it_be(:admin) { create(:admin) } before do diff --git a/spec/lib/gitlab/encoding_helper_spec.rb b/spec/lib/gitlab/encoding_helper_spec.rb index db7961fc0c9adf646c0f6a6bea6bc71a8c011919..57f7012c0b478937360d230063d9c9e8b32775f2 100644 --- a/spec/lib/gitlab/encoding_helper_spec.rb +++ b/spec/lib/gitlab/encoding_helper_spec.rb @@ -293,7 +293,6 @@ end # This cop's alternative to .dup doesn't work in this context for some reason. - # rubocop: disable Performance/UnfreezeString describe "#force_encode_utf8" do let(:stringish) do Class.new(String) do @@ -325,5 +324,4 @@ expect(described_class.force_encode_utf8(string)).to eq("bår") end end - # rubocop: enable Performance/UnfreezeString end