8000 Removing unused variable. by iluxonchik · Pull Request #496 · chocolatey/choco · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Removing unused variable. #496

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Removing unused variable. #496

wants to merge 1 commit into from

Conversation

iluxonchik
Copy link
Contributor

Removing unused local variable "count" in NugetService.cs

@ferventcoder
Copy link
Member

Are you familiar with our CONTRIBUTING document? Please read over it and adjust your commit message appropriately. This is probably a maint commit.

@iluxonchik
Copy link
Contributor Author

@ferventcoder whoops, my bad, somehow missed that part (about prefixing with the type of change). Will do.

@iluxonchik iluxonchik force-pushed the remove-unused-variable branch from 81c49ac to fd156d1 Compare December 1, 2015 15:35
@iluxonchik
Copy link
Contributor Author

@ferventcoder fixed the commit message. Is "Removing" acceptable or should it be "Remove"?

@ferventcoder
Copy link
Member

Remove. Good catch.

@iluxonchik
(maint) Remove unused variable
48cdada 
Remove unused local variable "count" in NugetService.cs
@iluxonchik iluxonchik force-pushed the remove-unused-variable branch from fd156d1 to 48cdada Compare December 1, 2015 16:43
@ferventcoder
Copy link
Member

Thanks for the fix!

@ferventcoder ferventcoder added this to the 0.9.10 milestone Dec 10, 2015
@ferventcoder ferventcoder self-assigned this Dec 10, 2015
@iluxonchik
Copy link
Contributor Author

@ferventcoder no problem :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@ferventcoder ferventcoder

Labels
5 - Released
Projects
None yet
Milestone
0.9.10
Development

Successfully merging this pull request may close these issues.
3 participants
@iluxonchik @ferventcoder @gep13
0