-
Notifications
You must be signed in to change notification settings - Fork 5
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
OPSEXP-1802 Pre-Commit Hooks Best Practices in Build Tools #192
Conversation
b0bcb9c
to
c45a03f
Compare
@gionn I have renamed the file to .md now. What else were you explaining me to check in order to avoid white spaces issue ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added a few inline comments, for a brief markdown introduction take a look at https://github.com/jorditorresBCN/Quick-Start/blob/master/Quick-Start-Markdown.md
Co-authored-by: Giovanni Toraldo <giovanni.toraldo@hyland.com>
Co-authored-by: Giovanni Toraldo <giovanni.toraldo@hyland.com>
@gionn Suggested changes applied. Please review again! |
Hi @alxgomz as Gio is out, could you please review and approve accordingly? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
your markdown code blocks for yaml are malformated.
Use ```yaml and start writing content on a new line
As Gio is on vacation so removed his review. |
Requested changed got addressed and as Gio is on vacation so dismissing his review.
No description provided.