Skip to content
This repository was archived by the owner on Jan 24, 2023. It is now read-only.

Remove Pry gem dependency #3

Merged
merged 5 commits into from
Sep 18, 2020
Merged

Remove Pry gem dependency #3

merged 5 commits into from
Sep 18, 2020

Conversation

wevtimoteo
Copy link
Member

This was adding pry gem in our production environment. Freezing pry version preventing us to use ruby-yard project.

  • Remove CC::CLI::Console
  • Remove pry dependency
  • Remove exception from Lint/Debugger
  • Bump ruby version to 2.6.5

@wevtimoteo wevtimoteo self-assigned this Sep 18, 2020
@sourcelevel-bot
Copy link

SourceLevel has finished reviewing this Pull Request and has found:

  • 2 possible new issues (including those that may have been commented here).
  • 63 fixed issues! 🎉

See more details about this review.

beatrizChagas
beatrizChagas approved these changes Sep 18, 2020
View reviewed changes
Copy link
Contributor

@beatrizChagas beatrizChagas left a comment

Choose a reason for hiding this comment

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

🎉

@wevtimoteo wevtimoteo merged commit a0cacb9 into master Sep 18, 2020
@wevtimoteo wevtimoteo deleted the wt-remove-pry-dependency branch September 18, 2020 20:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@garaujodev garaujodev garaujodev approved these changes

@beatrizChagas beatrizChagas beatrizChagas approved these changes

Assignees

@wevtimoteo wevtimoteo

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@wevtimoteo @garaujodev @beatrizChagas