Skip to content

Conversation

@agriyakhetarpal
Copy link
Member

This PR bumps the theme used to build the SPECs to the new v0.22 release, adapting new syntax for figure, image, and button shortcodes via scientific-python/scientific-python-hugo-theme#674. This is building upon @goanpeca's work in #394. Closes #394.

cc: @stefanv

@agriyakhetarpal agriyakhetarpal changed the title Bump SPHT to v0.22 + adopt new syntax for applicable shortcodes #887 Bump SPHT to v0.22 + adopt new syntax for applicable shortcodes Dec 12, 2025
@stefanv stefanv closed this Dec 12, 2025
@stefanv stefanv reopened this Dec 12, 2025
@agriyakhetarpal
Copy link
Member Author

Hmm, does either of you know why Netlify can't find yaml2ics on PATH?

@agriyakhetarpal
Copy link
Member Author

I thought it's related to scientific-python/scientific-python-hugo-theme@c975d87 at first, but it doesn't seem like it...

@bsipocz
Copy link
Member

bsipocz commented Dec 12, 2025

Hmm, does either of you know why Netlify can't find yaml2ics on PATH?

I have a very bad track record of figuring out netlify's hissy fits :(

@bsipocz
Copy link
Member

bsipocz commented Dec 12, 2025

(I would expect this should work just as well as for the tools repo, but apparently it's not the case)

@stefanv
Copy link
Member

stefanv commented Dec 12, 2025

Hmm, does either of you know why Netlify can't find yaml2ics on PATH?

Trying to recall exactly why: something to do with which binary is picked up when running pip and yaml2ics.

IIRC the fix is to invoke python -m yaml2ics in the Makefile. I made the change in some places already, but apparently not consistently throughout the ecosystem.

@stefanv
Copy link
Member

stefanv commented Dec 12, 2025

Ah, wait, this fix is already in place on scientific-python.org:

10:24:58 AM: ((python -c 'import yaml2ics' && pre-commit) > /dev/null 2>&1) || python -m pip install -q -r requirements.txt

@stefanv
Copy link
Member

stefanv commented Dec 12, 2025

Could try python -m yaml2ics.

@stefanv
Copy link
Member

stefanv commented Dec 12, 2025

Let's merge this and fix on scientific-python.org.

@stefanv stefanv merged commit 45fa637 into scientific-python:main Dec 12, 2025
1 of 2 checks passed
@agriyakhetarpal agriyakhetarpal deleted the new-theme-update branch December 12, 2025 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file maintenance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants