Skip to content

Conversation

@jetersen
Copy link
Contributor

@jetersen jetersen commented Feb 5, 2020

fixes #2082

@jetersen
Copy link
Contributor Author

jetersen commented Feb 5, 2020

This depends on #2077 to not flag line endings warning.

@jetersen
Copy link
Contributor Author

jetersen commented Feb 5, 2020

If you prefer I can convert it to an separate azure pipeline stage, see #2082

@arturcic
Copy link
Member

arturcic commented Feb 7, 2020

Makes sense to use this on PRs and only on GitHubAction, to validate PRs are well formatted.

@arturcic
Copy link
Member

arturcic commented Feb 7, 2020

@Casz can you fix the code review requested and I will merge

@jetersen
Copy link
Contributor Author

jetersen commented Feb 7, 2020

@arturcic perhaps you forgot to submit the review? I am not seeing any 😓

@arturcic
Copy link
Member

arturcic commented Feb 7, 2020

@arturcic perhaps you forgot to submit the review? I am not seeing any 😓

You're right, submitted now

Copy link
Contributor Author

@jetersen jetersen left a comment

Choose a reason for hiding this comment

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

WDYT @arturcic

@arturcic arturcic merged commit 5cc1e65 into GitTools:master Feb 7, 2020
@arturcic
Copy link
Member

arturcic commented Feb 7, 2020

thank you so much for your contributions 👍 @Casz

@jetersen jetersen deleted the actionFormat branch February 7, 2020 19:40
@jetersen
Copy link
Contributor Author

jetersen commented Feb 7, 2020

Thanks for accepting my contribution :)

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