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

Exclude files from templating based on .gitignore #20

Merged
merged 3 commits into from
Jun 7, 2020

Conversation

72636c
Copy link
Member

@72636c 72636c commented Jun 6, 2020

We used to exclude a hardcoded list of "filenames" from templating, so we didn't chug on trying to rewrite all of .git or node_modules. This easily fell out of sync and couldn't represent complex paths.

Instead, we use .gitignores as a basis for which files to exclude.

We used to exclude a hardcoded list of "filenames" from templating, so
we didn't chug on trying to rewrite all of `.git` or `node_modules`.
This easily fell out of sync and was couldn't represent complex paths.

Instead, we use `.gitignore`s as a basis for which files to exclude.
@72636c 72636c requested a review from a team as a code owner June 6, 2020 14:00
@changeset-bot
Copy link

changeset-bot bot commented Jun 6, 2020

🦋 Changeset is good to go

Latest commit: 3d7a0f1

We got this.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@@ -46,24 +46,3 @@ export const TEMPLATE_PROMPT = new Select({
message: 'Select a template:',
name: 'templateName',
});

export const IGNORED_TEMPLATE_FILENAMES = new Set([
Copy link
Contributor

Choose a reason for hiding this comment

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

🥇

@72636c 72636c merged commit 50a316f into master Jun 7, 2020
@72636c 72636c deleted the gitignore-templating branch June 7, 2020 04:34
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.

None yet

2 participants