- 
                Notifications
    You must be signed in to change notification settings 
- Fork 570
feat(pkg/protoanalysis): ignore hidden files #1296
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
Conversation
4cf9dd2    to
    4cc9197      
    Compare
  
    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.
Looks nice! 🦖
e738bdf    to
    1c5a8d6      
    Compare
  
    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.
Let's make sure that integration tests are passing. 👍
| 
 Thanks for the help. I'm facing some problems running the integration tests in the mac m1. I need to figure out a way for me can run it locally | 
| Yeah, integration tests are pretty slow. Specially on Rosetta it might be longer. 😄 What works for me is, I only run the test I need on my local and let the CI run the rest of them for me. | 
a089c0d    to
    485d096      
    Compare
  
    485d096    to
    dc35eab      
    Compare
  
    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.
Perfect! 🍾
* improve glob feature and also exclude hidden folders from the path walk * fix the integration tests
Closes #1249
What does this PR does?
mattn/go-zglobpackage from thego.mod;xospackage methods;How to test?
$ go test github.com/tendermint/starport/starport/pkg/localfs -run Test_glob -v