Skip to content

CLI #1

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
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

CLI #1

wants to merge 1 commit into from

Conversation

indikaimk
Copy link

Added a CLI.

@joeljunstrom
Copy link
Owner

Nice feature!

Unless I'm missing something I think you accidentally added some temporary files from your editor (ending with ~)

Could you remove them and squash?

@indikaimk
Copy link
Author

Oh!... I am still new to Ruby. Can you tell me where this method get_input_number is.

@indikaimk indikaimk closed this Jun 16, 2012
@indikaimk indikaimk reopened this Jun 16, 2012
@indikaimk
Copy link
Author

I will remove the temporary file and upload ASAP

@joeljunstrom
Copy link
Owner

Regarding the get_input_number, in the validate method you use the get_input_number method, but in the calculate method you seem to reimplement it?

@indikaimk
Copy link
Author

Oops..I will correct it and upload. That has been a silly mistake...

@joeljunstrom
Copy link
Owner

No worries mate, just make a new commit and squash your branch and then force push it.

http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@indikaimk @joeljunstrom