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

Add stack implementation with stack.py and readme.me #15

Merged
merged 2 commits into from
Oct 2, 2024

Conversation

Charul00
Copy link
Contributor

@Charul00 Charul00 commented Oct 2, 2024

Pull Request for PyVerse 💡
Issue Title: Implement Stack Data Structure
Info about the related issue (Aim of the project): This project aims to implement a stack data structure using Python, providing essential operations such as push, pop, peek, and is_empty. The stack will be demonstrated through a practical implementation, enhancing understanding of LIFO (Last In First Out) behavior in data structures.

Name: Charul Chim

GitHub ID: Charul00

Email ID: charulchim18@gmail.com

Identify yourself: gssoc-ext contributor

Closes: #14

Describe the add-ons or changes you've made 📃
In this pull request, I have created a stack folder that contains:

stack.py: This file implements the stack data structure with methods to perform push, pop, peek, and check if the stack is empty.
README.md: This file provides an overview of the stack data structure, its properties, and a guide on how to use the implemented code. It also includes explanations for the various methods in the stack.py file, making it easier for users to understand the implementation.
Type of change ☑️
What sort of change have you made:

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Code style update (formatting, local variables)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
This change requires a documentation update
How Has This Been Tested? ⚙️
The stack implementation has been tested by performing a series of push and pop operations to ensure the Last In First Out (LIFO) property is maintained. Each method has been invoked with various inputs to validate functionality, and the results have been printed for verification.

Checklist: ☑️
My code follows the guidelines of this project.
I have performed a self-review of my own code.
I have commented my code, particularly wherever it was hard to understand.
I have made corresponding changes to the documentation.
My changes generate no new warnings.
I have added things that prove my fix is effective or that my feature works.
Any dependent changes have been merged and published in downstream modules.

Copy link

github-actions bot commented Oct 2, 2024

👋 Thank you for opening this pull request! We're excited to review your contribution. Please give us a moment, and we'll get back to you shortly!

@github-actions github-actions bot requested a review from UTSAVS26 October 2, 2024 06:47
@UTSAVS26
Copy link
Owner

UTSAVS26 commented Oct 2, 2024

@Charul00 please add the correct issue

@Charul00
Copy link
Contributor Author

Charul00 commented Oct 2, 2024

@Charul00 please add the correct issue

i have added check please

@TheChaoticor
Copy link
Collaborator

add a test/example for checking if and how the stack works for different data types

Copy link
Collaborator

@TheChaoticor TheChaoticor left a comment

Choose a reason for hiding this comment

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

add a test/example for checking if and how the stack works for different data types

@Charul00
Copy link
Contributor Author

Charul00 commented Oct 2, 2024

add a test/example for checking if and how the stack works for different data types

in readme file right?

@TheChaoticor
Copy link
Collaborator

no,in the code. Add a simple test/example of atleast two different data types and try with strings too and comment its output

@Charul00
Copy link
Contributor Author

Charul00 commented Oct 2, 2024

no,in the code. Add a simple test/example of atleast two different data types and try with strings too and comment its output

done please check now

@TheChaoticor TheChaoticor merged commit c5be6d4 into UTSAVS26:main Oct 2, 2024
@TheChaoticor
Copy link
Collaborator

merged,good work

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

📃: Stacks
3 participants