Skip to content

Fix ToC building from H2's when the H2 has an extra tag#698

Open
brent-dxw wants to merge 2 commits intodevfrom
fix/15382-h2-bold-headings-issue
Open

Fix ToC building from H2's when the H2 has an extra tag#698
brent-dxw wants to merge 2 commits intodevfrom
fix/15382-h2-bold-headings-issue

Conversation

@brent-dxw
Copy link
Copy Markdown
Collaborator

dragon-dxw and others added 2 commits May 27, 2022 17:51
Docker build and CI were broken with the error message:

Step 5/31 : RUN apk add closure-compiler --update-cache --repository http://dl-3.alpinelinux.org/alpine/edge/testing/ --allow-untrusted &&     rm -f /tmp/* /etc/apk/cache/* /root/.cache
 ---> Running in 550968f4fc23
fetch http://dl-3.alpinelinux.org/alpine/edge/testing/x86_64/APKINDEX.tar.gz
ERROR: unable to select packages:
  closure-compiler (no such package):
    required by: world[closure-compiler]

I'm not entirely sure what the effects of removing this package and no longer cleaning the javascript with
it are: I suspect it is not critical -- increased javascript size, potentially missing a few optimisations.

There are links in Google search to https://pkgs.alpinelinux.org/package/edge/testing/x86/closure-compiler
with text suggesting a v20171203-r1 version existed, but the page itself 404s. A link does exist in the
AUR (https://aur.archlinux.org/packages/closure-compiler)

Some notes on next steps to try to reintegrate closure-compiler at #702 (comment)
…ag such as bold

- The ticket specifically highlights issue if the heading is both H2 and bold.
- This will only handle one extra tag with the H2 element

- Resolves: https://dxw.zendesk.com/agent/tickets/15382
@brent-dxw brent-dxw force-pushed the fix/15382-h2-bold-headings-issue branch from a29ceda to 41f5ad0 Compare May 30, 2022 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