Skip to content

Conversation

@tomoasleep
Copy link
Member

@tomoasleep tomoasleep commented May 14, 2024

This PR adds graphql-ruby 2.x to available versions.

Also, this PR prepares multiple gemfiles to test with graphql-ruby 1.x and 2.x.
These gemfiles are generated by appraisal.

@qlty-cloud-legacy
Copy link

qlty-cloud-legacy bot commented May 14, 2024

Code Climate has analyzed commit 7bf51ae and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 100.0%.

View more on Code Climate.

@tomoasleep tomoasleep force-pushed the support-graphql-2 branch from 5c82f65 to 7bf51ae Compare May 14, 2024 08:22
@tomoasleep tomoasleep force-pushed the support-graphql-2 branch 3 times, most recently from d9bd6e2 to 6298325 Compare December 17, 2024 12:52
@tomoasleep tomoasleep changed the base branch from master to update-rubocop-rspec December 17, 2024 12:52
@tomoasleep tomoasleep changed the base branch from update-rubocop-rspec to master December 17, 2024 12:57
Comment on lines +1 to +4
# Merge default configurations on Exclude option
inherit_mode:
merge:
- Exclude
Copy link
Member Author

Choose a reason for hiding this comment

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

RuboCop has default Exclude configurations (link).
With this option, the default configurations and the configurations in this file are merged.

Copy link
Member Author

@tomoasleep tomoasleep Dec 18, 2024

Choose a reason for hiding this comment

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

Without this option, RuboCop on CI may fail due to lack of default Exclude configurations. (RuboCop reads configuration files and ruby files in vendor/bundle/**/*)

Ref: rubocop/rubocop#9832

@tomoasleep tomoasleep requested a review from Umekawa December 18, 2024 12:05
Copy link
Member

@Umekawa Umekawa left a comment

Choose a reason for hiding this comment

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

LGTM

@tomoasleep tomoasleep merged commit 8de3b6c into master Dec 19, 2024
8 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