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: do not modify index.ejs with default viewport meta tag #1693

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Den-dp
Copy link

@Den-dp Den-dp commented Oct 20, 2021

Take into account src/index.ejs existence while deciding to add default meta-tags.

Closes: #1387

@jantimon
Copy link
Owner

I really like this pr 👍. Right now html-webpack-plugin 5 tries to introduce no breaking changes to stay aligned with webpack 5.

Unfortunately this pr might break peoples sites..

.gitignore Outdated Show resolved Hide resolved
@stale
Copy link

stale bot commented Apr 27, 2022

This issue had no activity for at least half a year. It's subject to automatic issue closing if there is no activity in the next 15 days.

@stale stale bot added the wontfix label Apr 27, 2022
Take into account `src/index.ejs` existence while deciding to add default meta-tags.

Closes: jantimon#1387
@Den-dp Den-dp force-pushed the fix/do-not-modify-index.ejs-with-default-meta-tag branch from 4cb3def to 26a3704 Compare June 4, 2022 16:15
@stale stale bot removed the wontfix label Jun 4, 2022
@Den-dp
Copy link
Author

Den-dp commented Jun 4, 2022

I've rebased my PR just in case

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

An extra ‘viewport’ meta with default template 'src/index.ejs'
3 participants