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

remove largely useless Utils class #135

Merged
merged 1 commit into from Jan 27, 2017
Merged

remove largely useless Utils class #135

merged 1 commit into from Jan 27, 2017

Conversation

lamont-granquist
Copy link
Contributor

its just a collection of private methods, they should be private methods

@coderanger
Copy link
Contributor

Sooooo I'm actually using one of those :-/ https://github.com/poise/poise/blob/master/lib/poise/utils/win32.rb#L116

@lamont-granquist
Copy link
Contributor Author

lol, 'k

@coderanger
Copy link
Contributor

Works for me, I can switch over my code to do a feature detect on it easily enough so we can nuke the stub eventually.

@lamont-granquist
Copy link
Contributor Author

doesn't pass in your tests right now, i'm sure i've just got some thinko/typo somewhere...

@lamont-granquist
Copy link
Contributor Author

oic what the issue is...

its just a collection of private methods, they should be private methods

retain a method in the class that was being used externally.

Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
@lamont-granquist
Copy link
Contributor Author

@coderanger fixed and squashed, passes your tests, can i get a 👍 ?

Copy link
Contributor

@coderanger coderanger left a comment

Choose a reason for hiding this comment

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

:shipit:

@lamont-granquist lamont-granquist merged commit e901532 into master Jan 27, 2017
@lamont-granquist lamont-granquist deleted the lcg/cleanup1 branch January 27, 2017 23:33
@chef chef locked and limited conversation to collaborators Nov 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants