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

Fet: Missing Postfix Evaluation In Stack section #4360

Closed
5 tasks done
BamaCharanChhandogi opened this issue Sep 9, 2023 · 4 comments
Closed
5 tasks done

Fet: Missing Postfix Evaluation In Stack section #4360

BamaCharanChhandogi opened this issue Sep 9, 2023 · 4 comments
Labels
enhancement good first issue

Comments

@BamaCharanChhandogi
Copy link
Member

BamaCharanChhandogi commented Sep 9, 2023 •
edited

What would you like to Propose?

What would you like to Propose?

  • I have read CONTRIBUTING.md.
  • This pull request is all my own work -- I have not plagiarized it.
  • All filenames are in PascalCase.
  • All functions and variable names follow Java naming conventions.
  • All new algorithms have a URL in their comments that points to Wikipedia or other similar explanations.

Issue details

Move Stack Postfix Notation algo into the Stack section.

Additional Information

No response
@siriak
Copy link
Member

siriak commented Sep 9, 2023

Is this what you wanted? https://github.com/TheAlgorithms/Java/blob/c54b8cddf38dec124a4b23b4512ea363286d8570/src/main/java/com/thealgorithms/others/StackPostfixNotation.java

@BamaCharanChhandogi
Copy link
Member Author

BamaCharanChhandogi commented Sep 9, 2023 •
edited

Yes, this file I wanted.
but, why is this file located in the Others section? It should be in the Stack section.
I saw many files are located in different sections.

@siriak
Copy link
Member

siriak commented Sep 10, 2023

Hmm, I guess you could move it to Stack section, this makes sense

@BamaCharanChhandogi BamaCharanChhandogi changed the title Fet:Postfix Evaluation Fet: Missing Postfix Evaluation In Stack section Sep 12, 2023
@beradeep beradeep mentioned this issue Sep 13, 2023
Merged
5 tasks
@beradeep
Copy link
Contributor

I think this issue can be closed now

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

No branches or pull requests
3 participants
@siriak @BamaCharanChhandogi @beradeep