Skip to content
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 CONTAINS #7

Merged
merged 1 commit into from
Jan 11, 2016
Merged

ADD CONTAINS #7

merged 1 commit into from
Jan 11, 2016

Conversation

xinity
Copy link

@xinity xinity commented Jan 3, 2016

No description provided.

@l0rd
Copy link
Owner

l0rd commented Jan 6, 2016

This PR include also the commits of PR #6. Can you please rebase this branch with master ?

@xinity
Copy link
Author

xinity commented Jan 8, 2016

almost finished :)
need to get ride of "grep -q ! " instead of "! grep -q" and will be fine

@l0rd
Copy link
Owner

l0rd commented Jan 8, 2016

Great! Thanks @xinity

@xinity
Copy link
Author

xinity commented Jan 11, 2016

all right, all fixed except i haven't found how to get rid of "grep -q !" to make it "! grep -q''

i'll update this PR anyway, if you have any clues please let me know.

sorry not having not enough GO skills :(

@l0rd
Copy link
Owner

l0rd commented Jan 11, 2016

No problem for the grep -q ! I will fix it in a separate commit. Thanks for your contribution @xinity!

l0rd added a commit that referenced this pull request Jan 11, 2016
Implement FILE_CONTAINS template
@l0rd l0rd merged commit 8620c02 into l0rd:master Jan 11, 2016
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