Skip to content

Extend api copilot#22

Merged
briandowns merged 2 commits into
masterfrom
extend_api_copilot
Jul 21, 2026
Merged

Extend api copilot#22
briandowns merged 2 commits into
masterfrom
extend_api_copilot

Conversation

@briandowns

Copy link
Copy Markdown
Owner

No description provided.

Signed-off-by: Brian Downs <brian.downs@gmail.com>
Signed-off-by: Brian Downs <brian.downs@gmail.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR adjusts build/install OS branching in the Makefile, adds an additional (newer) GitHub API version header constant, and updates the sample program to demonstrate a different API call (page views metrics).

Changes:

  • Simplified Darwin vs non-Darwin shared library targets and aligned install/uninstall library paths to $(LIBDIR).
  • Added a second API version header macro and corrected an error-message allocation size in gh_client_code_of_conduct_get_by_key.
  • Switched example.c to execute the page-views metrics call instead of the repositories list example.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.

File Description
Makefile Simplifies OS conditionals and fixes install/uninstall library removal paths.
github.c Adds an extra API-version macro and tweaks an error-message allocation size.
example.c Changes which API call the example runs (page views metrics).
docs/.keep Placeholder file to keep the docs directory present.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread github.c
Comment thread example.c
@briandowns
briandowns merged commit 34d140b into master Jul 21, 2026
3 of 7 checks passed
@briandowns
briandowns deleted the extend_api_copilot branch July 21, 2026 02:13
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