Skip to content

Deduplicate Linter comments #595

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 9 commits into from
Closed

Deduplicate Linter comments #595

wants to merge 9 commits into from

Conversation

mattnite
Copy link
Contributor

No description provided.

@mattnite mattnite changed the title dedupe comments Deduplicate Linter comments Jun 13, 2025
@@ -847,3 +847,5 @@ inline fn custom_find_import_pkg_hash_or_fatal(comptime dep_name: []const u8) []

@panic("dependency not found");
}

fn myFunc() void {}
Copy link
Collaborator

Choose a reason for hiding this comment

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

was this for testing?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yup, not seeing the github comment, for some reason the commits for head and the branch in the github action are the same

Copy link
Collaborator

Choose a reason for hiding this comment

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

is HEAD the branch in this context? vs. main?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I thought it was supposed to be the head of main, but it's probably just the branch. I iterated a few times on this with an LLM and it seems to be falling short. I'll actually go read the docs.

@mattnite mattnite force-pushed the linter-dedupe-comments branch from 55ea0c8 to fc04bbc Compare June 22, 2025 22:48
@mattnite mattnite closed this Jun 23, 2025
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.

2 participants