-
Notifications
You must be signed in to change notification settings - Fork 66
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
Add rules diff verbosity flag #199
Add rules diff verbosity flag #199
Conversation
@pracucci status update? make images is failing for my forked repo locally so relying on this repo for production ticket |
built docker image locally from my forked repo but still not seeing diff rules. hoping to get this merged here this week if possible? @jtlisi @gotjosh @gouthamve |
Thank you very much for your contribution @justinTM, this is on my list for this week. |
@gotjosh not to spam but any status update? if we could set up a super quick 15 minute call maybe i can get local docker image built with your help? i wouldn't mind writing short documentation afterwards for contributors to build locally, too. |
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.
LGTM
Signed-off-by: Goutham Veeramachaneni <gouthamve@gmail.com>
The second part even though has |
@justinTM Can you open an issue for the docker images with clear steps and the errors you were seeing? This'll help us debug the issue and if its not obvious, I'll be happy to jump on a call with you! |
* add rules diff verbosity flag * Make the deleted lines - Signed-off-by: Goutham Veeramachaneni <gouthamve@gmail.com> Co-authored-by: Justin Mai <justin.mai@drillinginfo.com> Co-authored-by: Goutham Veeramachaneni <gouthamve@gmail.com>
No description provided.