Skip to content

docs: 🏗️ add check command pseudocode - #133

Open
lwjohnst86 wants to merge 5 commits into
mainfrom
docs/add-check-cli-pseudocode
Open

docs: 🏗️ add check command pseudocode#133
lwjohnst86 wants to merge 5 commits into
mainfrom
docs/add-check-cli-pseudocode

Conversation

@lwjohnst86

@lwjohnst86 lwjohnst86 commented Aug 28, 2026

Copy link
Copy Markdown
Member

Description

This is potentially how the check CLI will work, though I'm not sure how the printing of Errors will work or look like. I think we'll have a better idea once we start implementing things and testing things out.

Related to #130, closes #115

Needs a thorough review.

Checklist

  • Ran just run-all

@lwjohnst86 lwjohnst86 moved this from Todo to In Review in Product development Aug 28, 2026

@joelostblom joelostblom 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.

Small comments again

Comment thread src/cli.rs Outdated
Comment thread src/cli.rs Outdated
Comment thread src/cli.rs Outdated
@github-project-automation github-project-automation Bot moved this from In Review to In Progress in Product development Sep 1, 2026

@DanMazJen DanMazJen 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.

Nice work! 🎉
I added some small comments and questions I had while reading 😄

Comment thread src/cli.rs Outdated
Comment thread src/cli.rs Outdated
Comment thread src/cli.rs Outdated
@lwjohnst86 lwjohnst86 moved this from In Progress to In Review in Product development Sep 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

Draft (mostly placeholder) for check CLI

3 participants