Skip to content

Conversation

lwjohnst86
Copy link
Member

Description

Closes #123

This PR needs an in-depth review.

Checklist

  • Formatted Markdown
  • Ran just run-all

@lwjohnst86 lwjohnst86 requested a review from a team as a code owner July 25, 2025 15:06
@lwjohnst86 lwjohnst86 moved this from Todo to In Review in Iteration planning Jul 25, 2025
Copy link
Contributor

@martonvago martonvago left a comment

Choose a reason for hiding this comment

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

Nice, just some comments ☺️

@github-project-automation github-project-automation bot moved this from In Review to In Progress in Iteration planning Jul 28, 2025
Copy link
Member

@signekb signekb left a comment

Choose a reason for hiding this comment

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

Nice! Just some minor suggestions ✏️

docs/guide.qmd Outdated
Comment on lines 13 to 15
- [Python](https://www.python.org/): Required for installing and using
many of the tools in this template, and because the template
itself uses Python.
many of the tools in this template, and because the template itself
uses Python.
Copy link
Member

Choose a reason for hiding this comment

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

Maybe have that the template itself uses Python first? That must be the primary reason.

lwjohnst86 and others added 2 commits July 28, 2025 10:58
Co-authored-by: martonvago <57952344+martonvago@users.noreply.github.com>
Co-authored-by: Signe Kirk Brødbæk <40836345+signekb@users.noreply.github.com>
@lwjohnst86 lwjohnst86 requested review from martonvago and signekb July 28, 2025 09:05
@lwjohnst86 lwjohnst86 moved this from In Progress to In Review in Iteration planning Jul 28, 2025
Copy link
Contributor

@martonvago martonvago left a comment

Choose a reason for hiding this comment

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

Just a typo

@github-project-automation github-project-automation bot moved this from In Review to In Progress in Iteration planning Jul 28, 2025
signekb
signekb previously approved these changes Jul 28, 2025
Copy link
Member

@signekb signekb left a comment

Choose a reason for hiding this comment

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

Except the typo.

Co-authored-by: martonvago <57952344+martonvago@users.noreply.github.com>
@lwjohnst86 lwjohnst86 requested review from martonvago and signekb July 28, 2025 11:21
@lwjohnst86 lwjohnst86 enabled auto-merge (squash) July 28, 2025 11:21
@lwjohnst86 lwjohnst86 disabled auto-merge July 28, 2025 11:24
@lwjohnst86 lwjohnst86 merged commit 4b80d29 into main Jul 28, 2025
5 checks passed
@lwjohnst86 lwjohnst86 deleted the docs/add-guide-for-updating branch July 28, 2025 11:24
@github-project-automation github-project-automation bot moved this from In Progress to Done in Iteration planning Jul 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Guide for using in an existing repo

3 participants