Skip to content

ci: Validate JupyterBook build on PRs without deploying - #7

Open
atodorov wants to merge 2 commits into
fmipython:mainfrom
atodorov:ci/build-on-prs
Open

atodorov wants to merge 2 commits into
fmipython:mainfrom
atodorov:ci/build-on-prs

Conversation

@atodorov

@atodorov atodorov commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

За да знаем винаги, че HTML страниците на сайта може да се компилират (за хората които редактират файлове в терминала и допускат грешки).

@yalishanda42 yalishanda42 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a, nice

@yalishanda42 yalishanda42 left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

само дето сега се води фейл вместо просто да се скипват. Може би да се направи отделен джоб и на него да се сложи if-a? Ако не ти се занимава, ще затворя и отворя аз като имам време (понеже не вярвам да имам пуш права в твоя форк 😄 )

Build the book on every push and pull request so PRs validate the
build without touching GitHub Pages. Move the protected github-pages
environment, Pages permissions, and deploy-pages step into a separate
deploy job that only runs outside pull requests.

Manual workflow_dispatch runs still deploy as before.
@atodorov

atodorov commented Nov 9, 2026

Copy link
Copy Markdown
Contributor Author

само дето сега се води фейл вместо просто да се скипват. Може би да се направи отделен джоб и на него да се сложи if-a?

Сега ще пробвам, но трябва да го пуснеш да се изцикли, за да видим дали ще проработи.

понеже не вярвам да имам пуш права в твоя форк 😄

Ако виждаш ето това може и да имаш :-)
image

@atodorov
atodorov requested a review from yalishanda42 November 9, 2026 11:54

This branch had an error being deployed

1 failed (outdated) deployment
github-pages — 7ab953c3 Deployed Nov 8, 2026 by atodorov via render #22
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