Skip to content
This repository was archived by the owner on Apr 5, 2022. It is now read-only.

Conversation

@AlicanAkkus
Copy link

No description provided.

@pivotal-issuemaster
Copy link

@AlicanAkkus Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-issuemaster
Copy link

@AlicanAkkus Thank you for signing the Contributor License Agreement!

@AlicanAkkus
Copy link
Author

@spencergibb hi, can you review pr and can you share your opinions with me? thanks.

@spencergibb
Copy link
Contributor

@dsyer would be the one to review.

@AlicanAkkus
Copy link
Author

@dsyer would be the one to review.

I'm waiting for your review :)

@dsyer
Copy link

dsyer commented Jun 1, 2019

LGTM. Lots of whitespace changes though. Maybe we should add the spring-format plugin.

@AlicanAkkus
Copy link
Author

AlicanAkkus commented Jun 1, 2019

LGTM. Lots of whitespace changes though. Maybe we should add the spring-format plugin.

I've added the format plugin and did reformat.

@AlicanAkkus
Copy link
Author

@dsyer @spencergibb hi, can you review again, please?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants