Add breadth first search by Rolv-Apneseth · Pull Request #3632 · TheAlgorithms/Python · GitHub
Skip to content

Add breadth first search - #3632

Closed
Rolv-Apneseth wants to merge 4 commits into
TheAlgorithms:masterfrom
Rolv-Apneseth:add_breadth_first_search
Closed

Rolv-Apneseth wants to merge 4 commits into
TheAlgorithms:masterfrom
Rolv-Apneseth:add_breadth_first_search

Conversation

@Rolv-Apneseth

Copy link
Copy Markdown
Contributor

Added a .py file in the searches folder for breadth-first search algorithm with a pathfind visualisation to show the result.

Included doctests and have already run black, mypy and doctests locally.

In relation to #3260

@Rolv-Apneseth

Copy link
Copy Markdown
Contributor Author

@kadhirash

Copy link
Copy Markdown

Hello @Rolv-Apneseth,

I'm thinking they both ended up counting as commits because you pushed commits from (#3454) earlier, regardless of what branch you are on currently. To get rid of the old branch changes, you have to delete them locally and remotely I believe, or stash them in case you would want to come back to it.

Essentially you want to remove your Git changes.

Anyway, after this, the old branch commits wouldn't be part of your commits I'm thinking.

@poyea

poyea commented Oct 22, 2020

Copy link
Copy Markdown
Member

Please run black . and isort for your code. https://github.com/TheAlgorithms/Python/blob/master/CONTRIBUTING.md

@poyea

poyea commented Oct 22, 2020

Copy link
Copy Markdown
Member

I think you can enable the github actions in your setting of the fork too.

@poyea

poyea commented Oct 22, 2020

Copy link
Copy Markdown
Member

You may open a new branch for those changes, and file a new PR. When you've done, you can close this PR.

@Rolv-Apneseth

Copy link
Copy Markdown
Contributor Author

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.

3 participants