Skip to content

Fix engine typo: knitter → knitr in terrain and movement tutorials#126

Merged
petrasovaa merged 1 commit into
OSGeo:mainfrom
Valyrian-Code:fix/engine-knitter-typo
Jun 3, 2026
Merged

Fix engine typo: knitter → knitr in terrain and movement tutorials#126
petrasovaa merged 1 commit into
OSGeo:mainfrom
Valyrian-Code:fix/engine-knitter-typo

Conversation

@Valyrian-Code

Copy link
Copy Markdown
Contributor

Four tutorials had engine: knitter (misspelling) instead of engine: knitr. Quarto does not recognise knitter as a valid engine and silently falls back to a default.

Affected files:

  • terrain_and_DEMs/GRASS_terrain.qmd
  • terrain_and_DEMs/GRASS_terrain_pt.qmd
  • modeling_movement/GRASS_movement.qmd
  • modeling_movement/GRASS_movement_pt.qmd

The same typo in remote_sensing_visualization/GRASS_remotesensing.qmd is addressed separately in PR #125.

Copilot AI review requested due to automatic review settings June 3, 2026 04:57

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Fixes a YAML front-matter typo in several Quarto tutorial files so the intended execution engine is correctly specified.

Changes:

  • Replaced engine: knitter with engine: knitr in multiple .qmd tutorials.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
content/tutorials/terrain_and_DEMs/GRASS_terrain_pt.qmd Corrects Quarto engine setting typo in YAML front matter.
content/tutorials/terrain_and_DEMs/GRASS_terrain.qmd Corrects Quarto engine setting typo in YAML front matter.
content/tutorials/modeling_movement/GRASS_movement_pt.qmd Corrects Quarto engine setting typo in YAML front matter.
content/tutorials/modeling_movement/GRASS_movement.qmd Corrects Quarto engine setting typo in YAML front matter.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@petrasovaa petrasovaa merged commit a966dc2 into OSGeo:main Jun 3, 2026
1 check passed
@Valyrian-Code Valyrian-Code deleted the fix/engine-knitter-typo branch June 3, 2026 17:43
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.

3 participants