Skip to content

GH-51008: [C++] Remove vendored whereami - #54024

Merged
pitrou merged 1 commit into
apache:mainfrom
Reranko05:remove-whereami
Oct 19, 2026
Merged

pitrou merged 1 commit into
apache:mainfrom
Reranko05:remove-whereami

Conversation

@Reranko05

@Reranko05 Reranko05 commented Oct 17, 2026 •

Copy link
Copy Markdown
Collaborator

Rationale for this change

Remove the unused vendored whereami library and its associated license text from LICENSE.txt.

What changes are included in this PR?

  • Remove the vendored whereami source and header files.
  • Remove the associated license text from LICENSE.txt.

Are there any user-facing changes?

No

Was AI used for this PR?

In accordance to the AI generation guidelines, please disclose below whether and how AI was used in this PR.

PR code and description written by:

  • Human
  • AI

Reviewed before submission by:

  • Human
  • AI
  • Not reviewed

Closes: #51008

Copilot AI balanced review requested due to automatic review settings October 17, 2026 04:31

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

Copy link
Copy Markdown

⚠️ GitHub issue #51008 has been automatically assigned in GitHub to PR creator.

@github-actions github-actions Bot added awaiting committer review Awaiting committer review and removed awaiting review Awaiting review labels Oct 19, 2026
@pitrou
pitrou merged commit fb767a7 into apache:main Oct 19, 2026
63 of 65 checks passed
@pitrou pitrou removed the awaiting committer review Awaiting committer review label Oct 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[C++] Remove vendored whereami

3 participants