Skip to content

API Audit and Holistic Review#63

Merged
agl-alexglopez merged 11 commits intomainfrom
api-audit
Nov 30, 2025
Merged

API Audit and Holistic Review#63
agl-alexglopez merged 11 commits intomainfrom
api-audit

Conversation

@agl-alexglopez
Copy link
Copy Markdown
Collaborator

Interface naming, formatting, and internal implementations get a holistic review as I try to ensure this makes it to a version 1.0 soon.

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

clang-tidy made some suggestions

There were too many comments to post at once. Showing the first 10 out of 52. Check the log or trigger a new build to see more.

Comment thread samples/mini_grep.c
Comment thread samples/mini_grep.c
Comment thread str_view/str_view.h
Comment thread str_view/str_view.h
Comment thread str_view/str_view.h
Comment thread str_view/str_view.h
Comment thread str_view/str_view.h
Comment thread str_view/str_view.h Outdated
Comment thread str_view/str_view.h Outdated
Comment thread str_view/str_view.h Outdated
@agl-alexglopez agl-alexglopez marked this pull request as ready for review November 30, 2025 21:10
@agl-alexglopez agl-alexglopez merged commit 75f797d into main Nov 30, 2025
7 checks passed
@agl-alexglopez agl-alexglopez deleted the api-audit branch November 30, 2025 21:10
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