Skip to content

add blog and images#455

Open
DCEW wants to merge 18 commits into
mainfrom
add_brain_plates_blog
Open

add blog and images#455
DCEW wants to merge 18 commits into
mainfrom
add_brain_plates_blog

Conversation

@DCEW

@DCEW DCEW commented May 28, 2026

Copy link
Copy Markdown
Contributor

The Quarto workflows started failing after switching away from the Quarto render action: CI had R available, but not the knitr / rmarkdown packages required to render posts with frozen R content. This change restores the repo’s intended non-executing CI render path and installs only the R packages Quarto needs to parse those documents.

  • Workflow behavior

    • Keep both preview and publish workflows on quarto render . --execute false
    • Remove the split between trusted and fork PR render paths so CI follows one render mode consistently
  • R runtime dependencies

    • Add r-lib/actions/setup-r-dependencies@v2 to install the minimal packages Quarto needs:
      • any::knitr
      • any::rmarkdown
  • Contributor guidance

    • Update the README note on CI behavior to reflect that Actions now provides a minimal R setup for rendering, while computation still needs to happen locally and be committed via _freeze

Example workflow change:

- name: Setup R dependencies
  uses: r-lib/actions/setup-r-dependencies@v2
  with:
    extra-packages: |
      any::knitr
      any::rmarkdown

- name: Render
  run: quarto render . --execute false

@DCEW DCEW self-assigned this May 28, 2026
@DCEW DCEW marked this pull request as draft May 28, 2026 14:02
@github-actions

github-actions Bot commented May 28, 2026

Copy link
Copy Markdown
PR Preview Action v1.4.8
🚀 Deployed preview to https://The-Strategy-Unit.github.io/data_science/pr-preview/pr-455/
on branch gh-pages at 2026-06-22 17:11 UTC

@StatsRhian StatsRhian removed their request for review June 1, 2026 09:37
Comment thread blogs/posts/2026-05-28_brain_plates/index.qmd
@DCEW DCEW marked this pull request as ready for review June 17, 2026 08:10
@DCEW DCEW requested a review from ai-mindset June 18, 2026 08:38
Copilot AI changed the title add blog and images Enable Quarto execution for trusted renders while preserving fork PR safety Jun 22, 2026
@ai-mindset ai-mindset changed the title Enable Quarto execution for trusted renders while preserving fork PR safety add blog and images Jun 22, 2026
Copilot AI changed the title add blog and images Fix Quarto Actions rendering from the workflow directory Jun 22, 2026
@ai-mindset ai-mindset changed the title Fix Quarto Actions rendering from the workflow directory add blog and images Jun 22, 2026
Copilot AI changed the title add blog and images Stabilize Quarto CI by installing minimal R render dependencies Jun 22, 2026
@tomjemmett

Copy link
Copy Markdown
Member

Didn't I create a PR a while back to purposely remove the need for R and its dependencies? Or was that a different repo...

@ai-mindset ai-mindset changed the title Stabilize Quarto CI by installing minimal R render dependencies add blog and images Jun 22, 2026
@ai-mindset

ai-mindset commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Didn't I create a PR a while back to purposely remove the need for R and its dependencies? Or was that a different repo...

You probably did. I was experimenting with GitHub Copilot but I'm not happy with the outcome. Will revert back to cd5cf14.

Edit: @tomjemmett how do pages get rendered without R?

@ai-mindset ai-mindset force-pushed the add_brain_plates_blog branch from 989604a to cd5cf14 Compare June 22, 2026 14:34
@tomjemmett

Copy link
Copy Markdown
Member

any computations should be committed to the _freeze directory (render local). otherwise, you should not need to render.

there are some caveats, see the readme. some things like code-folding seem to require rmarkdown and knitr to be available for no good reason, so it's best to avoid those features

Comment thread blogs/posts/2026-05-22-review-etiquette-with-gh-copilot/index.qmd
@ai-mindset

Copy link
Copy Markdown
Contributor

any computations should be committed to the _freeze directory (render local). otherwise, you should not need to render.

there are some caveats, see the readme. some things like code-folding seem to require rmarkdown and knitr to be available for no good reason, so it's best to avoid those features

The cuplrit most likely was a forgotten

```{r lockfile}
...
renv::use(lockfile = "renv.lock")
```

I hadn't deleted when I updated our Using Git and GitHub page. code-fold: true wasn't relevant here.

@ai-mindset ai-mindset requested a review from tomjemmett June 22, 2026 17:14
@tomjemmett tomjemmett removed their request for review June 24, 2026 08:00
@DCEW

DCEW commented Jun 24, 2026

Copy link
Copy Markdown
Contributor Author

Not sure what to do with this one now TBH given the confusion - will let @tomjemmett approve this PR

@tomjemmett tomjemmett left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@DCEW not read any of the content, so not really in a position to approve/disprove, but it doesn't seem to be borking the deployment anymore

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.

Add blog on how team members manage their "brain plates" (tasks, what they're responsible for)

3 participants