Skip to content

Translated some src/java programs #201

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

Closed
wants to merge 0 commits into from
Closed

Translated some src/java programs #201

wants to merge 0 commits into from

Conversation

BenEklundCS
Copy link
Contributor

Translated:

BuscaBinaria --> BinarySearch
Exponenciacao --> Exponentiation
ExponenciacaoRecursiva --> RecursiveExponentiation
TorreDeHanoi --> TowersOfHanoi

@BenEklundCS
Copy link
Contributor Author

#182 Helping here

@BenEklundCS
Copy link
Contributor Author

I'm not 100% sure if the old files are supposed to be renamed, or left as is?

Looking for help on my next steps. Thanks! :)

Copy link
Owner

@kelvins kelvins left a comment

Choose a reason for hiding this comment

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

Thanks @BenEklundCS 🎉 . Yes, we can rename/translate all files to english. I would suggest opening a PR only for renaming them because it would be easier to review.

Could you please fix the CI issues? Looks like the java linter is broken as well as the link checker.

Copy link
Owner

Choose a reason for hiding this comment

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

Could you please remove this .class file?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

In trying to edit my PR, I may have Closed it accidentally. I was going to remove my commits. How did this end up causing a force push?

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.

None yet

2 participants