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

Added new functionality to be able to cancel a search and to set a maximum number of subfolders to search in #17

Merged

Conversation

ericf-axosoft
Copy link
Contributor

@ericf-axosoft ericf-axosoft commented Sep 17, 2024

Summary of changes

  • Added maxSubfolderDeep parameter.
  • Modified the callback to be able to return true to cancel the search.
  • Fix compiling node module with spaces in path.
  • Updated readme file.
  • Added new tests.
  • Fixed GitHub action for the tests.

@ericf-axosoft ericf-axosoft force-pushed the GK-6252-add-new-parameters branch 6 times, most recently from 303ef64 to 7fa015d Compare September 17, 2024 11:32
@ericf-axosoft ericf-axosoft force-pushed the GK-6252-add-new-parameters branch from 7fa015d to 6e8b62b Compare September 17, 2024 14:06
@ericf-axosoft
Copy link
Contributor Author

@ianhattendorf could you please take a look at this PR?
I'm not able to add reviewers.

Thanks!

@ericf-axosoft ericf-axosoft force-pushed the GK-6252-add-new-parameters branch from 6e8b62b to c9ebe0b Compare September 17, 2024 14:14
@ianhattendorf ianhattendorf self-requested a review September 17, 2024 15:51
@ericf-axosoft
Copy link
Contributor Author

@ianhattendorf I also don't know why only one of the test nodes has been executed. The others are waiting all the time.

@ericf-axosoft ericf-axosoft force-pushed the GK-6252-add-new-parameters branch from 8113625 to 2298343 Compare September 18, 2024 09:33
@ianhattendorf ianhattendorf merged commit 2030b09 into Axosoft:master Sep 18, 2024
15 checks passed
@ericf-axosoft ericf-axosoft deleted the GK-6252-add-new-parameters branch September 19, 2024 08:05
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