Skip to content
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

refactor: Install header for private/experimental API functions #2453

Merged
merged 1 commit into from
Dec 7, 2023

Conversation

JFreegman
Copy link
Member

@JFreegman JFreegman commented Dec 5, 2023

This allows client developers to use the private API without needing to manually specify the header path in the source code, which may vary from system to system.


This change is Reviewable

@JFreegman JFreegman added packaging Packaging refactor Refactoring production code, eg. renaming a variable, not affecting semantics labels Dec 5, 2023
@JFreegman JFreegman added this to the v0.2.19 milestone Dec 5, 2023
@JFreegman JFreegman force-pushed the tox_private_install branch 2 times, most recently from 8e5f80b to 2fe3369 Compare December 6, 2023 00:58
@JFreegman JFreegman closed this Dec 6, 2023
@JFreegman JFreegman reopened this Dec 6, 2023
CMakeLists.txt Outdated Show resolved Hide resolved
@JFreegman JFreegman force-pushed the tox_private_install branch 3 times, most recently from f9cf77e to aea676e Compare December 7, 2023 17:12
Copy link

codecov bot commented Dec 7, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (d89677f) 74.32% compared to head (7a6d50e) 74.39%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2453      +/-   ##
==========================================
+ Coverage   74.32%   74.39%   +0.06%     
==========================================
  Files          87       87              
  Lines       26264    26264              
==========================================
+ Hits        19522    19538      +16     
+ Misses       6742     6726      -16     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@JFreegman JFreegman marked this pull request as ready for review December 7, 2023 18:01
This allows client developers to use the private API without
needing to manually specify the header path in the source
code, which may vary from system to system.
@JFreegman JFreegman merged commit 7a6d50e into TokTok:master Dec 7, 2023
53 checks passed
@JFreegman JFreegman deleted the tox_private_install branch December 7, 2023 19:31
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packaging Packaging refactor Refactoring production code, eg. renaming a variable, not affecting semantics
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants