Skip to content

Conversation

issyl0
Copy link
Contributor

@issyl0 issyl0 commented Apr 6, 2025

warning: Inheriting from `RuboCop::Cop::Cop` is deprecated. Use `RuboCop::Cop::Base` instead.
For more information, see https://docs.rubocop.org/rubocop/v1_upgrade_notes.html

An error occurred while GraphQL/Heredoc cop was inspecting /Users/issyl0/repos/github/graphql-client/test/test_client.rb:724:19.
unknown keyword: :location

issyl0 added 4 commits April 6, 2025 14:14
- This was showing a deprecation warning with the latest RuboCop version.

```
warning: Inheriting from `RuboCop::Cop::Cop` is deprecated. Use `RuboCop::Cop::Base` instead.
For more information, see https://docs.rubocop.org/rubocop/v1_upgrade_notes.html
``
```
Scanning /Users/issyl0/repos/github/graphql-client/test/test_client.rb
An error occurred while GraphQL/Heredoc cop was inspecting /Users/issyl0/repos/github/graphql-client/test/test_client.rb:724:19.
unknown keyword: :location
/Users/issyl0/.rbenv/versions/3.3.2/lib/ruby/gems/3.3.0/gems/rubocop-1.73.2/lib/rubocop/cop/base.rb:201:in `add_offense'
/Users/issyl0/repos/github/graphql-client/lib/rubocop/cop/graphql/heredoc.rb:22:in `block in check_str'
```
Copy link

@AlenaSviridenko AlenaSviridenko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👋
Thanks for reaching out!
To be honest, I am far from expert in this project, but the update does seem fine, so let's try :)

Copy link
Contributor

@jmeridth jmeridth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These changes make sense. Going to approve and merge ignoring current test failures. We can address the current test failures after.

@jmeridth jmeridth merged commit fa6df6f into github-community-projects:master May 28, 2025
1 of 81 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants