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

Creating a custom template with .png file breaks BYO templates #1454

Open
relientm96 opened this issue Feb 22, 2024 · 6 comments
Open

Creating a custom template with .png file breaks BYO templates #1454

relientm96 opened this issue Feb 22, 2024 · 6 comments

Comments

@relientm96
Copy link

relientm96 commented Feb 22, 2024

Description

I was trying to create a template with image/.png files in the README, but came across this error:

image

I assume that there is a <% somewhere in the raw version of the .png file but it isn't ignored by skuba when running skuba configure

I also realised that it does not run pnpm install neither does it also initialise your project as a git repository. So I think the configure step just fails.

@AaronMoat
Copy link
Contributor

Is the raw file safe to attach here in OSS @relientm96?

@relientm96
Copy link
Author

Is the raw file safe to attach here in OSS @relientm96?

Yeap the file is safe to be shared but unsure how to share the actual raw contents

Here is a link to the repo that I was trying to skuba configure that has the image: https://github.com/SEEK-Jobs/ap-2024-worker-template/blob/master/images/image.png

@AaronMoat
Copy link
Contributor

Yep:

$ cat Downloads/image.png | grep '<%'
Binary file (standard input) matches

@relientm96
Copy link
Author

Yep:

$ cat Downloads/image.png | grep '<%'
Binary file (standard input) matches

Ahh should have though of that, thanks for checking @AaronMoat 😆

@relientm96 relientm96 changed the title Creating a custom template with a .png file breaks BYO templates Creating a custom template with .png file breaks BYO templates Feb 22, 2024
@AaronMoat
Copy link
Contributor

AaronMoat commented Feb 22, 2024

Any thoughts on resolution? Skip the templating if it's in an exclusion list of file formats? Or perhaps inclusion list?

@relientm96
Copy link
Author

Any thoughts on resolution? Skip the templating if it's in an exclusion list of file formats? Or perhaps inclusion list?

Some sort of exclusion input on skuba-configure would be great as users of skuba would assume everything is included for now.

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

No branches or pull requests

2 participants