Skip to content

Add checkbox for storing viewed files in review - #21

Merged
ReeceStevens merged 12 commits into
mainfrom
add-checkbox-for-files
May 22, 2026
Merged

Add checkbox for storing viewed files in review#21
ReeceStevens merged 12 commits into
mainfrom
add-checkbox-for-files

Conversation

@ReeceStevens

Copy link
Copy Markdown
Owner

No description provided.

@ReeceStevens ReeceStevens left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

I'm placing a temporary review here to test pulling reviews from github

Comment thread src/lib.rs
api::command("sign define PrReviewComment text=C> texthl=Search linehl=DiffText")?;
api::command("highlight default PrReviewCommentSign guifg=#bf5af2 ctermfg=141")?;
api::command("sign define PrReviewCommentTop text=╭ texthl=PrReviewCommentSign")?;
api::command("sign define PrReviewCommentMid text=│ texthl=PrReviewCommentSign")?;

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Adding a test comment here

Comment thread src/lib.rs
let end_line = comment.line;
api::out_write(string!(
"{:?}: {}-{}\n",
buffer,

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Another test comment here (on the left side)

Comment thread src/lib.rs
side,
orphaned,
..
}) => {

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Longer range comment

import glob

print("Hello!")

Comment thread src/lib.rs
r#"syn match reviewStatusFile /^\s\+[MADRC?]\s.\+|/ contains=reviewStatusModifier,reviewStatusPath,reviewStatusPipe"#,
// File entry: " [x] M src/lib.rs | +45 -12"
r#"syn match reviewStatusFile /^\s\+\[.\]\s[MADRC?]\s.\+|/ contains=reviewStatusViewed,reviewStatusUnviewed,reviewStatusModifier,reviewStatusPath,reviewStatusPipe"#,
// Viewed/unviewed checkbox

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Nice styling here

@ReeceStevens

Copy link
Copy Markdown
Owner Author

Tests are passing locally-- not sure what the source of the compiler error is in CI , but for now I'm moving forward with merge and will resolve that in a subsequent ticket.

@ReeceStevens
ReeceStevens merged commit 07f39a0 into main May 22, 2026
2 of 3 checks passed
@ReeceStevens
ReeceStevens deleted the add-checkbox-for-files branch May 22, 2026 17:42
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.

1 participant