Skip to content

feat(vite-plugin): support glob for project files - #834

Merged
aarthificial merged 1 commit into
motion-canvas:mainfrom
rmhartog:support-glob-files
Oct 31, 2023
Merged

feat(vite-plugin): support glob for project files#834
aarthificial merged 1 commit into
motion-canvas:mainfrom
rmhartog:support-glob-files

Conversation

@rmhartog

Copy link
Copy Markdown
Contributor

Adopting the stale pull request #576 because it failed CI.

Support glob patterns for project files in the vite-plugin.

Please credit @guifeliper as below if squash-merging.

Closes: #324
Co-Authored-By: Guilherme Reis guifeliper@gmail.com

@aarthificial

Copy link
Copy Markdown
Contributor

Can we use fast-glob instead of glob?

@rmhartog

Copy link
Copy Markdown
Contributor Author

Can we use fast-glob instead of glob?

Looks like that'd be a simple replacement, will pick that up soon.

@rmhartog

Copy link
Copy Markdown
Contributor Author

Updated to fast-glob, let me know what you think @aarthificial :)

@aarthificial

Copy link
Copy Markdown
Contributor

Could you also update the TSDoc comments and the documentation to mention the support for globs?

Closes: motion-canvas#324

Co-Authored-By: Guilherme Reis <guifeliper@gmail.com>
@rmhartog

Copy link
Copy Markdown
Contributor Author

Could you also update the TSDoc comments and the documentation to mention the support for globs?

Do you think mentioning support like I did for now is sufficient?

@aarthificial
aarthificial merged commit 67029c4 into motion-canvas:main Oct 31, 2023
@aarthificial

Copy link
Copy Markdown
Contributor

Yes, thanks!

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.

Support glob for project files

2 participants