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.

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

postfix-expression-evaluation #61

Closed
TheSTL opened this issue Oct 10, 2019 · 13 comments
Closed

postfix-expression-evaluation #61

TheSTL opened this issue Oct 10, 2019 · 13 comments
Assignees
@k88manish
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@TheSTL
Copy link
Member

TheSTL commented Oct 10, 2019

File location postfix-expression-evaluation

If too few number arguments are passed then it should throw error or expression is not in order.
Few cases:

  • 12 Error
  • 1 Error
  • + Error
  • 1+2 Error
  • +12 Error
@TheSTL TheSTL added bug Something isn't working good first issue Good for newcomers Hacktoberfest labels Oct 10, 2019
@jasbindar-singh
Copy link

How do I run the project on my local machine?

@TheSTL
Copy link
Member Author

TheSTL commented Oct 10, 2019

Clone this repo and then run command npm install.

@jasbindar-singh
Copy link

That will just install the dependencies, then?

@TheSTL
Copy link
Member Author

TheSTL commented Oct 10, 2019

Then, node [file_path_to_run]

@vobango
Copy link

vobango commented Oct 11, 2019

If nobody wants this, I can start working on it on Monday.

@TheSTL
Copy link
Member Author

TheSTL commented Oct 11, 2019

Hi @vobango , we are not getting your commit properly . Please setup you git properly.

@ashokdey
Copy link
Member

How do I run the project on my local machine?

Hello @jasbindar-singh, there is nothing to set up, it's a dead-simple project.

@jasbindar-singh
Copy link

How do I run and see outputs can you elaborate. @ashokdey

@TheSTL
Copy link
Member Author

TheSTL commented Oct 15, 2019

@vobango are you working on this issue?

@k88manish
Copy link
Contributor

k88manish commented Oct 17, 2019 •
edited
Loading

@TheSTL If no one is working on it can I take this up?

@TheSTL
Copy link
Member Author

TheSTL commented Oct 17, 2019

Yes you can work on this @k88manish

@k88manish k88manish mentioned this issue Oct 20, 2019
Closed
@k88manish
Copy link
Contributor

@TheSTL Can you review my PR #118

k88manish added a commit to k88manish/problem-solving-javascript that referenced this issue Oct 23, 2019
@k88manish
postfix-expression-evaluation knaxus#61
430e7f3 
- Update test case for exception
@k88manish k88manish mentioned this issue Oct 23, 2019
Merged
TheSTL added a commit that referenced this issue Oct 23, 2019
@TheSTL
Merge pull request #129 from k88manish/bug-fix-postfix-expression
1aaa484 
postfix-expression-evaluation #61
@TheSTL
Copy link
Member Author

TheSTL commented Oct 23, 2019

I'm closing this issue.
Merged PR is -[#129 ]

@TheSTL TheSTL closed this as completed Oct 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@k88manish k88manish

Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@ashokdey @vobango @k88manish @jasbindar-singh @TheSTL