Bump external-content/cookie from 4d7762b to 228add5#319
Bump external-content/cookie from 4d7762b to 228add5#319dependabot[bot] wants to merge 4 commits into
4d7762b to 228add5#319Conversation
Bumps [external-content/cookie](https://github.com/scientific-python/cookie) from `4d7762b` to `228add5`. - [Release notes](https://github.com/scientific-python/cookie/releases) - [Commits](scientific-python/cookie@4d7762b...228add5) --- updated-dependencies: - dependency-name: external-content/cookie dependency-version: 228add5438e25e3357d0c3f9afc088ce48df25f4 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
dependabot
Bot
added
dependencies
✅ Deploy Preview for learn-scientific-python-org ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Cookie moved its docs from Jekyll to MyST in scientific-python/cookie#792. Build the development site with `myst build --html` (BASE_URL=/development) and drop the obsolete ruby/jekyll and repo-review-app fetch steps. Assisted-by: ClaudeCode:claude-opus-4.8
Use a trailing slash on BASE_URL so MyST joins it to root-relative page links correctly (was producing /developmentpages/...), and bump the cookie submodule to include the hyphenated-slug internal link fix (scientific-python/cookie#796) so checklinks passes. Assisted-by: ClaudeCode:claude-opus-4.8
netlify-plugin-checklinks (hyperlink) now traverses the full cookie site and flags two non-issues: its parser fails on MyST's bundled mermaid chunk, and it reports valid in-page fragments as broken when the target goes through MyST pretty-URL normalization. Mark both as todoPatterns so real broken links still fail the build. Assisted-by: ClaudeCode:claude-opus-4.8
|
🤖 AI text below 🤖 CI is green. Deploy Preview ready! and all checks pass (the "skipping" entries are Netlify's optional rules-changed checks, not failures). Done — CI fixedThe deploy preview now builds and passes link-checking: https://deploy-preview-319--learn-scientific-python-org.netlify.app It took fixing three layers, since each was masking the next:
Plus the upstream cookie fix you merged (#796): the guide's internal links used underscore filenames ( To verify before pushing, I reproduced One note worth flagging: cookie's docs have a number of genuinely-broken external links (404s on |
|
A newer version of external-content/cookie exists, but since this PR has been edited by someone other than Dependabot I haven't updated it. You'll get a PR for the updated version as normal once this PR is merged. |
1 participant
Bumps external-content/cookie from
4d7762bto228add5.Commits
228add5chore(deps): bump versions (#794)d8a8d57feat: move to mystmd (#792)cc6c89cchore(deps): bump astral-sh/setup-uv from 8.1.0 to 8.2.0 in the actions group...Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)