Skip to content

Notebook serialization fix + CI skips docs/notebooks - #16

Merged
HolyWill90 merged 1 commit into
mainfrom
chore/ci-paths
Sep 17, 2026
Merged

HolyWill90 merged 1 commit into
mainfrom
chore/ci-paths

Conversation

@HolyWill90

Copy link
Copy Markdown
Owner

Two fixes, one commit:

  1. The notebook rendered as single-line garbage in Colab — cell sources were stored as newline-split lists without trailing newlines, which nbformat joins directly. All 8 cells are now plain multiline strings and verified per cell. This is what corrupted the first cell into bash syntax errors on your run.
  2. CI now skips the suite for docs/markdown/notebook-only changes — they cannot affect the Rust binaries, and a notebook tweak does not justify 15 minutes of runners. This PR touches only a notebook and the workflow, so it skips CI by its own rule.

The notebook's cell sources were stored as newline-split lists without
trailing newlines, which nbformat joins into one line — every cell
rendered as single-line garbage in Colab. Sources are now plain
multiline strings (verified per cell).

CI now skips the 10-job suite when only docs, markdown, or notebooks
change — they cannot affect a single line of Rust, and a notebook
tweak does not deserve 15 minutes of runners. This PR is itself
docs+notebook-only, so it skips CI by its own rule.
@HolyWill90
HolyWill90 merged commit 5535d66 into main Sep 17, 2026
10 checks passed
@HolyWill90
HolyWill90 deleted the chore/ci-paths branch September 17, 2026 03:51
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.

1 participant