Skip to content

Make the jupyterlite_contents glob recursive. - #72

Merged
martinRenou merged 1 commit into
jupyterlite:mainfrom
jasongrout:recursive
Aug 16, 2022
Merged

Make the jupyterlite_contents glob recursive.#72
martinRenou merged 1 commit into
jupyterlite:mainfrom
jasongrout:recursive

Conversation

@jasongrout

@jasongrout jasongrout commented Aug 16, 2022

Copy link
Copy Markdown
Collaborator

Since we have shifted to using globs for the contents, we might as well enable the strictly more powerful ** recursive syntax. See https://docs.python.org/3/library/glob.html#glob.glob

"If recursive is true, the pattern “**” will match any files and zero or more directories, subdirectories and symbolic links to directories. If the pattern is followed by an os.sep or os.altsep then files will not match."

Since we have shifted to using globs for the contents, we might as well
enable the strictly more powerful ** recursive syntax. See https://docs.python.org/3/library/glob.html#glob.glob

"If recursive is true, the pattern “**” will match any files and zero or more directories, subdirectories and symbolic links to directories. If the pattern is followed by an os.sep or os.altsep then files will not match."

@martinRenou martinRenou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you!

@martinRenou
martinRenou merged commit 52b0af6 into jupyterlite:main Aug 16, 2022
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