Remove files orphaned by the 11ty retirement - #5872
Open
ZJvandeWeg wants to merge 2 commits into
Open
ZJvandeWeg wants to merge 2 commits into
ZJvandeWeg wants to merge 2 commits into
Conversation
Nothing references any of these since Nuxt took over every route: - 254 assets under nuxt/public (old blog/handbook images, the pre-docs application-guide images, unused pictograms, etc.) - 62 nav-icons SVGs that nuxt/utils/navIcons.ts never imports - UseCaseLinks, IconsArrowRightIcon and IconsCarIcon components - roadmap.css, zoom-style.css and style.animations.css, with their imports - isSearchPage in lib/search-index.js, which read 11ty collection items CODEOWNERS drops the non-existent handbook/development entry and points the customer-success rule at its current path under sales/. Unreferenced files that could still be linked from outside the site (/images root files and /whitepaper images) are kept for a manual check.
No source imports it and no storage config uses the netlify-blobs driver; the build cache uses unstorage's fs-lite driver.
👷 Deploy Preview for flowfuse-website processing.
|
This branch has not been deployed
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.
Description
Follow-up to #5795. Now that Nuxt serves every route, these files have no references left:
nuxt/public/: old blog and handbook images (including the 12.8 MBhandbook/images/git-how-toGIFs), the images from before the Application Guide moved under/docs/, and unused pictogramsnuxt/assets/nav-icons/thatnuxt/utils/navIcons.tsnever importsuse-case/Links.vue,icons/ArrowRightIcon.vue,icons/CarIcon.vueroadmap.css,zoom-style.cssandstyle.animations.css, plus their@imports (the page they styled is gone, the library isn't installed, and the classes aren't used)isSearchPageinlib/search-index.js, which read 11ty collection items@netlify/blobs: nothing imports it and nothing configures thenetlify-blobsstorage driverAlso fixed:
CODEOWNERS: dropped the entry forhandbook/development, which doesn't exist, and pointed the customer-success rule at its current path undersales/.claude/CLAUDE.md: the icon note now says onlypulseandunsexist in both icon foldersEach asset was checked for references by full path, relative path, URL-encoded path and bare filename. The search covered this repo, the
flowfuse/flowfusedocs and the blueprint library, and allowed for paths built at build time (home logos, hero config, team headshots).Not included: 27 unreferenced files in the
/images/root and/whitepaper/images/, such as the HubSpot newsletter header and the logo wordmarks. They could be linked from emails or other sites, so they need checking by hand first.npm testpasses and the PostCSS build still compiles. I haven't run a full Nuxt build locally, so the deploy preview is the check for that.Related Issue(s)
#5795
Checklist