8000 Cleanup monkey patches by avdv · Pull Request #409 · athityakumar/colorls · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Cleanup monkey patches #409

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

Merged
merged 3 commits into from
Nov 2, 2020
Merged

Cleanup monkey patches #409

merged 3 commits into from
Nov 2, 2020

Conversation

avdv
Copy link
Collaborator
@avdv avdv commented Nov 2, 2020

Description

This removes some left-over code which is no longer needed, and also cleans up some monkey patch methods.

  • Relevant Issues : (none)
  • Relevant PRs : (none)
  • Type of change :
    • New feature
    • Bug fix for existing feature
    • Code quality improvement
    • Addition or Improvement of tests
    • Addition or Improvement of documentation

avdv and others added 3 commits November 2, 2020 08:45
* delete `#remove` method
* remove `#delete_prefix` spec, it was only used for Ruby < 2.5 (see 86cfa2f)
* add specifications for `#colorize` and `#uniq`
Usage in a single place does not really warrant a monkey patch and it
can be replaced with `transform_keys!` easily.
```
The `--path` flag is deprecated because it relies on being remembered across bundler invocations, which bundler will no longer do in future versions. Instead please use `bundle config set path vendor/bundle`, and stop using this flag
```
@avdv avdv merged commit b59484f into athityakumar:master Nov 2, 2020
@avdv avdv deleted the refactor-cleanup branch November 2, 2020 09:34
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
0