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

translation:Update chapter_stack_and_queue/stack.md #1038

Merged
merged 2 commits into from
Jan 28, 2024
Merged

translation:Update chapter_stack_and_queue/stack.md #1038

merged 2 commits into from
Jan 28, 2024

Conversation

K3v123
Copy link
Contributor

@K3v123 K3v123 commented Jan 10, 2024

did some minor changes.

If this PR is related to coding or code translation, please fill out the checklist and paste the console outputs to the PR.

  • I've tested the code and ensured the outputs are the same as the outputs of reference code.
  • I've checked the code (formatting, comments, indentation, file header, etc) carefully.
  • The code does not rely on a particular environment or IDE and can be executed on a standard system (Win, macOS, Ubuntu).

@K3v123
Update stack.md
ef3e6b1 
did some minor changes
@K3v123 K3v123 changed the title Update stack.md translation:Update stack.md Jan 10, 2024
@krahets
Copy link
Owner

krahets commented Jan 10, 2024 •
edited
Loading

Randomly assign 2 reviewers ['@yanedie' '@Phoenix0415'] to the pull request #1038

@krahets krahets added translation English translation documents documents-related labels Jan 10, 2024
@krahets krahets changed the title translation:Update stack.md translation:Update chapter_stack_and_queue/stack.md Jan 13, 2024
Phoenix0415
Phoenix0415 reviewed Jan 24, 2024
View reviewed changes
Copy link
Contributor

@Phoenix0415 Phoenix0415 left a comment

Choose a reason for hiding this comment

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

Great job on your work and thanks for the PR! I've only left one comment for you to consider.

docs-en/chapter_stack_and_queue/stack.md Outdated Show resolved Hide resolved
@K3v123
Update stack.md
83f3f18 
changed the part to what Phoenix has suggested.
@krahets krahets merged commit b8b4b09 into krahets:main Jan 28, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Phoenix0415 Phoenix0415 Phoenix0415 approved these changes

@yanedie yanedie yanedie approved these changes

@krahets krahets krahets approved these changes

Assignees

@K3v123 K3v123

Labels
documents documents-related translation English translation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@K3v123 @krahets @yanedie @Phoenix0415