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

Suppress warnings when application/javascript is used [0.2] #2621

Closed
antross opened this issue Jun 19, 2019 · 0 comments · Fixed by #2633
Closed

Suppress warnings when application/javascript is used [0.2] #2621

antross opened this issue Jun 19, 2019 · 0 comments · Fixed by #2633

Comments

@antross
Copy link
Member

antross commented Jun 19, 2019

Although text/javascript is the current best practice per discussion in #568 and noted in the HTML standard, the practical value of switching from application/javascript to text/javascript is relatively low.

More importantly this recommendation continues to cause confusion within the community as application/javascript was previously recommended by the IETF which hasn't yet updated RFC 4329 to recommend text/javascript (though an active draft exists which will make that change).

Given the conflicting state of currently published standards, the level of confusion this causes, and the limited practical benefit of using one of these mime types over the other, I propose changing hint-content-type to not raise any issues when application/javascript is used.

Note the recommended type would remain text/javascript with this change and warnings would still occur with this recommendation when any other mime-type is used (e.g. application/x-javascript).

Also note that users would still be able to override the new behavior in .hintrc to require a single type of their choosing if desired.

@antross antross changed the title Suppress warnings when application/javascript is used by default Suppress warnings when application/javascript is used Jun 19, 2019
@antross antross added this to the 1907-1 milestone Jun 21, 2019
@antross antross self-assigned this Jun 24, 2019
@antross antross changed the title Suppress warnings when application/javascript is used Suppress warnings when application/javascript is used [0.2] Jul 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant