Tighten types and remove IE-specific cruft in faces.js#5729
Open
BalusC wants to merge 3 commits into
Open
Conversation
BalusC
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#5506 and jakartaee/faces#1598
Goal: enforce strict tsconfig:
And remove now unnecessary IE-specific fluff (4.x compiled for ES5, 5.0 compiles for ES2020 which won't run in any IE in first place).
Requires jakartaee/faces#2155 in build path in order to build/test.
1 test has been dropped (IE fallback in error checking in faces.push) and all 362 original tests on faces.js pass; then 24 new tests have been added to tighten the test coverage.