Skip to content

Conversation

@chrischrischris
Copy link
Contributor

Needed for adobe/da-live#243

Previously crawl takes a root dir (path arg) and crawls everything within that dir and its children. Now we can provide an array of paths to crawl, as well as a list of files to also crawl.

@aem-code-sync
Copy link

aem-code-sync bot commented Nov 18, 2025

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

  • Re-run PSI checks
  • Re-sync branch
Commits

Copy link
Member

@auniverseaway auniverseaway left a comment

Choose a reason for hiding this comment

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

I think "files" is probably misleading since its actually folders you want to traverse.

I would suggest:

  1. paths (as opposed to path)
  2. folders

My only argument against paths is that its so similar to path but that may be a feature... if someone does paths + string, we could convert to an array. Same for path... if they provide an array of paths to path we can upgrade it to an array.

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.

3 participants