Opened 2 months ago
Last modified 6 weeks ago
#62463 new defect (bug)
The `precommit:emoji` script no longer works in branches 6.4 and earlier
Reported by: | johnbillion | Owned by: | |
---|---|---|---|
Milestone: | Awaiting Review | Priority: | normal |
Severity: | normal | Version: | |
Component: | Build/Test Tools | Keywords: | |
Focuses: | Cc: |
Description
In r57758 the precommit:emoji
script was updated to fetch the list of Twemoji files via a GraphQL query to the jdecked/twemoji
repo.
This script in branches 6.4 and earlier still tries to fetch the list from the trunk
branch of the twitter/twemoji
repo. The trunk
branch no longer exists, therefore this command fails.
r57758 should be backported.
Previously:
Change History (3)
#2
follow-up:
↓ 3
@
8 weeks ago
No I don't think there is, but it gets run as part of npm run grunt:precommit
.
#3
in reply to:
↑ 2
@
6 weeks ago
Replying to johnbillion:
No I don't think there is, but it gets run as part of
npm run grunt:precommit
.
Is it worth considering removing the emoji step from the older branches?
As people love emoji, it can serve as a motivation to update to the supported version of WP. Although this might make life hard fro the security team if an issue is found in the emoji library.
Are there any scenarios where the Twemoji library should be updated in these older branches?