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

refactor: polyfill zlib #185

Merged
merged 4 commits into from
Oct 8, 2024
Merged

Conversation

Ryuno-Ki
Copy link
Contributor

@Ryuno-Ki Ryuno-Ki commented Oct 7, 2024

This is tweaking the webpack config a little to allow for a comparison.

See also #184.

Signed-off-by: André Jaenisch [email protected]

@RoelN
Copy link
Contributor

RoelN commented Oct 8, 2024

@Ryuno-Ki This seems the PR to contain everything that #184 also has. Let's use this one, if you agree!

Following
https://github.com/Pomax/lib-font/#how-do-i-use-this-with-webpack I was
looking to where Webpack is used, given that the only reference is Vue
here. But its vue.config.js has a webpackConfig property, so I followed
a hunch…

Signed-off-by: André Jaenisch <[email protected]>
This is tweaking the webpack config a little to allow for a comparison.

Signed-off-by: André Jaenisch <[email protected]>
@Ryuno-Ki
Copy link
Contributor Author

Ryuno-Ki commented Oct 8, 2024

Why not both meme

I prefer to have a choice :)

Ryuno-Ki and others added 2 commits October 8, 2024 11:02
@RoelN
Copy link
Contributor

RoelN commented Oct 8, 2024

It worked on my machine!

A classic! 😆

Thanks @Ryuno-Ki All these green checkmarks are putting me in a good mood!

@RoelN RoelN merged commit de90c6e into Wakamai-Fondue:master Oct 8, 2024
1 check passed
@Ryuno-Ki
Copy link
Contributor Author

Ryuno-Ki commented Oct 8, 2024

Meme with Buzz Lightyear saying "I see green! Green everywhere!"

@Ryuno-Ki Ryuno-Ki deleted the zlib-pollyfilled branch October 8, 2024 09:08
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