Skip to content

Conversation

@Pantani
Copy link
Collaborator

@Pantani Pantani commented Jun 25, 2021

Closes #1249

What does this PR does?

  • Create our own path walk to find file pattern into recursive folders;
  • Exclude hidden folders from the path walk;
  • Remove the mattn/go-zglob package from the go.mod;
  • Remove unused xos package methods;

How to test?

  • Run test:
$ go test github.com/tendermint/starport/starport/pkg/localfs -run Test_glob -v
  • Run chain
$ go run github.com/tendermint/starport/starport/cmd/starport chain serve

@Pantani Pantani self-assigned this Jun 25, 2021
@Pantani Pantani requested review from fadeev, ilgooz and lumtis as code owners June 25, 2021 01:25
@Pantani Pantani force-pushed the feat/improve-glob-search branch 2 times, most recently from 4cf9dd2 to 4cc9197 Compare June 25, 2021 04:00
Copy link
Member

@ilgooz ilgooz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks nice! 🦖

@ilgooz ilgooz changed the title feat: improve the glob search feat(pkg/protoanalysis): ignore hidden files Jun 25, 2021
@Pantani Pantani force-pushed the feat/improve-glob-search branch from e738bdf to 1c5a8d6 Compare June 25, 2021 19:45
@Pantani Pantani requested a review from ilgooz June 25, 2021 19:49
Copy link
Member

@ilgooz ilgooz left a 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. 👍

@Pantani
Copy link
Collaborator Author

Pantani commented Jun 28, 2021

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

@ilgooz
Copy link
Member

ilgooz commented Jun 28, 2021

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.

@Pantani Pantani force-pushed the feat/improve-glob-search branch 2 times, most recently from a089c0d to 485d096 Compare June 29, 2021 03:45
@Pantani Pantani force-pushed the feat/improve-glob-search branch from 485d096 to dc35eab Compare June 29, 2021 03:53
@Pantani Pantani requested a review from ilgooz June 29, 2021 04:12
ilgooz
ilgooz previously approved these changes Jun 29, 2021
Copy link
Member

@ilgooz ilgooz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect! 🍾

lumtis
lumtis previously approved these changes Jul 5, 2021
@Pantani Pantani dismissed stale reviews from lumtis and ilgooz via e9d6521 July 5, 2021 19:36
@Pantani Pantani requested review from ilgooz and lumtis July 5, 2021 19:37
@ilgooz ilgooz merged commit d159d4d into develop Jul 6, 2021
@ilgooz ilgooz deleted the feat/improve-glob-search branch July 6, 2021 07:39
Jchicode pushed a commit to Jchicode/cli that referenced this pull request Aug 9, 2023
* improve glob feature and also exclude hidden folders from the path walk

* fix the integration tests
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.

cosmosgen: proto files inside .git are not ignored

4 participants