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

Implemented functionality to ignore the path when updating the search path #59

Open
wants to merge 2 commits into
base: master
from

Conversation

@vsantos-desenv
Copy link

vsantos-desenv commented Jun 26, 2020

Implemented functionality to ignore the path when updating the search path.

Just set the ignorepath property in boss.json as an example below.

"ignorepath": [
	"test",
	"tests",
	"samples"
]
@snakeice snakeice self-requested a review Jun 26, 2020
go.mod Outdated Show resolved Hide resolved
go.sum Outdated Show resolved Hide resolved
utils/librarypath/librarypath.go Outdated Show resolved Hide resolved
utils/librarypath/librarypath.go Outdated Show resolved Hide resolved
utils/librarypath/dprojUtil.go Outdated Show resolved Hide resolved
Vinicius Vieira dos Santos
Created the method to filter the array
@vsantos-desenv vsantos-desenv requested a review from snakeice Jun 29, 2020
result = utils.Filter(paths, func(s string) bool {
var result bool = false
for _, value := range ignoredPaths {
result = strings.Contains(strings.ToUpper(s), strings.ToUpper(value))

This comment has been minimized.

@snakeice

snakeice Jul 7, 2020

Member

I think using strings.HasPrefix is better for now, based on the relative path of the project folder.

@snakeice snakeice requested a review from viniciussanchez Jul 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.