Skip to content

add Bubble Sort on Java #3

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
Merged
merged 2 commits into from
Feb 4, 2019
Merged

add Bubble Sort on Java #3

merged 2 commits into from
Feb 4, 2019

Conversation

zzzzzhangyanqiu
Copy link
Contributor

No description provided.

@64json
Copy link
Member

64json commented Jan 31, 2019

Hi @suiyueranzly, thank you for your pull request! How do you think of putting a newline (\n) instead of a tab (\t) after every swap log? Otherwise, everything looks great.

Also, do you mind renaming the class from Code to Main and sending PR again? I just found out this server-side issue that the class name has to be Main regardless of the file name in order to run the visualization on the website. (You can test the code on Scratch Paper.) I will be fixing this issue in the next few days.
@zzzzzhangyanqiu
Copy link
Contributor Author

OK , i will sending PR again later
renaming the class from Code to Main and putting a newline (\n) inste…
28c8a23 
…ad of a tab (\t) in Brute Force\Bubble Sort
@64json 64json merged commit 82cb610 into algorithm-visualizer:master Feb 4, 2019
raiman264 pushed a commit to raiman264/algorithms that referenced this pull request Oct 2, 2019
@trekhleb
Merge pull request algorithm-visualizer#3 from PeterShershov/master
c8b3fb9 
small refactor in some sorting algorithms for better readability
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants
@zzzzzhangyanqiu @64json