Updated the .codeclimate.yml file#342
Conversation
| exclude_paths: | ||
| - vendor/**/* | ||
| - ZeroBin/**/* | ||
| - tests/**/* |
There was a problem hiding this comment.
What is the difference here between two ** and one *?
There was a problem hiding this comment.
I don't know exactly but if have to guess,then they both are pretty same.I will update them
There was a problem hiding this comment.
Please don't guess. You can't ask for something to be merged if you don't know what the code does.
There was a problem hiding this comment.
I researched on this thing...they (* & ** ) are both same and does same work in this case for sure
|
I think this is your third pull request making similar changes to the codeclimate file. You should be updating a single pull request, or at least closing the pull requests you open that are no longer valid. |
|
yes i will close other pull requests......I created this since you hinted me to remove a line which was not needed.Is there anything else that i should add ?? |
|
The new pull request is present at....#355 |
2 participants
Removed the tests directory as you recommended.
Updated the .codeclimate.yml file
Please review ...:)
#328