Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Ruby 2.7 warnings #24

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

miguel-botelho
Copy link

This PR fixes current warnings in the gem due to circular dependencies:
image

This happens due to require statements that are not needed. Besides that, also updated yard to the latest version and removed an unused variable.

Not sure if this project still has CI setup, but I ran the test suite with Ruby 2.5 and Ruby 2.7 and all tests passed:
Ruby 2.5:
Screenshot 2020-10-23 at 10 36 43

Ruby 2.7:
Screenshot 2020-10-23 at 10 34 29

Add ruby-2.7.1 to `travis.yml` file
Update yard to latest version
Remove circular dependency on different files
Remove unused variable
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.

1 participant