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

feat: added eulerian path to graph #533

Merged
merged 5 commits into from
Oct 3, 2023

Conversation

jeevanpranav02
Copy link
Contributor

@jeevanpranav02 jeevanpranav02 commented Oct 2, 2023

Description

Implementation of Eulerian Path from the issue #344

Type of change

  • New feature (non-breaking change which adds functionality)

Checklist:

  • I ran bellow commands using the latest version of rust nightly.
  • I ran cargo clippy --all -- -D warnings just before my last commit and fixed any issue that was found.
  • I ran cargo fmt just before my last commit.
  • I ran cargo test just before my last commit and all tests passed.
  • I checked COUNTRIBUTING.md and my code follows its guidelines.

I hope this closes the issue for Euclerian Path.
I would request if you can add label:hactoberfest-accepted to this PR if applicable. :)

@jeevanpranav02 jeevanpranav02 changed the title Graph feat: added eulerian path to graph Oct 2, 2023
@siriak
Copy link
Member

siriak commented Oct 2, 2023

Please check why cargo clippy fails

@jeevanpranav02
Copy link
Contributor Author

I checked the warning, and when I ran cargo clippy --all -- -D warnings there was no error on my PC. However, here, Clippy suggested a change, which I made. If there are any other changes you'd like me to make, please feel free to ask. I'll be happy to do it.

Copy link
Member

@siriak siriak left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@siriak siriak enabled auto-merge (squash) October 3, 2023 08:20
@siriak siriak merged commit 69c76a3 into TheAlgorithms:master Oct 3, 2023
@jeevanpranav02
Copy link
Contributor Author

Thanks @siriak Can you please add label:hactoberfest-accepted : )

@jeevanpranav02 jeevanpranav02 deleted the graph branch October 3, 2023 21:41
@jeevanpranav02 jeevanpranav02 restored the graph branch October 3, 2023 21:41
@jeevanpranav02 jeevanpranav02 deleted the graph branch October 3, 2023 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants