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

Fix: hidden files/directories were not included #15

Merged
merged 1 commit into from
Feb 28, 2024

Conversation

lfritz
Copy link
Contributor

@lfritz lfritz commented Feb 28, 2024

Files and directories in the template starting with . were not included anymore after this change:
#14

Files and directories in the template starting with . were not included
anymore after this change:
#14
@lfritz lfritz requested a review from katcipis February 28, 2024 08:52
@@ -13,8 +13,8 @@ import (
"text/template"
)

//go:embed templates/*
var templates embed.FS
//go:embed templates/basic/*
Copy link
Contributor Author

@lfritz lfritz Feb 28, 2024

Choose a reason for hiding this comment

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

When you're embedding a directory tree with go:embed, entries starting with a . are ignored unless they're named explicitly. The * includes entries starting with a . so that also counts as naming them explicitly.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It looks like the motivation for implementing it this way was to avoid the .DS_Store files that Mac OS tends to litter its filesystem with: golang/go#42328
Thanks, Apple!

Copy link
Contributor

@katcipis katcipis Feb 28, 2024

Choose a reason for hiding this comment

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

auheuaheaueh I remember this because I had some issues with dotfiles not being embedded here, but I did miss it on the review, sorry (I dont remember it that well...I just think I was bitten by this more than once 😆 ).

Fucking Bradley/Apple !!!

Copy link
Contributor

Choose a reason for hiding this comment

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

It also ignores underscore...didn't remember that 😆

"all the files to embed will ignore names beginning with dot or underscore, the same as the go command does
for deciding what to build."

why the fuck underscore ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure, maybe some editors create temporary files with names starting with underscore?

Copy link
Contributor

Choose a reason for hiding this comment

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

ugh...probably 😆

@lfritz lfritz merged commit 464f4de into main Feb 28, 2024
2 checks passed
@lfritz lfritz deleted the fix-missing-hidden-files branch February 28, 2024 15:25
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