Skip to content

Bob approaches#2236

Merged
ErikSchierboom merged 12 commits into
exercism:mainfrom
bobahop:bob-approaches
Jan 24, 2023
Merged

Bob approaches#2236
ErikSchierboom merged 12 commits into
exercism:mainfrom
bobahop:bob-approaches

Conversation

@bobahop
Copy link
Copy Markdown
Member

@bobahop bobahop commented Jan 23, 2023

pull request

Approaches for the Bob exercise


Reviewer Resources:

Track Policies

@bobahop
Copy link
Copy Markdown
Member Author

bobahop commented Jan 23, 2023

@ErikSchierboom : Java Bob approaches ready for review. I put code into an online Java code formatter and checked against coding conventions

Copy link
Copy Markdown
Member

@ErikSchierboom ErikSchierboom left a comment

Choose a reason for hiding this comment

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

Just one indentation comment

Comment thread exercises/practice/bob/.approaches/if-statements/snippet.txt Outdated
Co-authored-by: Erik Schierboom <erik_schierboom@hotmail.com>
@bobahop
Copy link
Copy Markdown
Member Author

bobahop commented Jan 24, 2023

@ErikSchierboom: indentation suggestion accepted. Thanks for the catch! Ready for merge.

@ErikSchierboom ErikSchierboom merged commit 512d048 into exercism:main Jan 24, 2023
@bobahop bobahop deleted the bob-approaches branch January 24, 2023 12:30
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.

2 participants