Skip to content

Conversation

@dixonjoel
Copy link
Collaborator

What does this Pull Request accomplish?

Add ni.measurementlink.pinmap.v1.proto so we can use it in another repo and remove the duplicated stubs there.

Why should this Pull Request be merged?

Single source stubs from pin_map_service.proto.

What testing has been done?

Will publish and use the package after submission.

@github-actions
Copy link
Contributor

github-actions bot commented Aug 20, 2025

Test Results

   46 files  ±0     46 suites  ±0   30s ⏱️ -1s
  144 tests ±0    144 ✅ ±0  0 💤 ±0  0 ❌ ±0 
1 424 runs  ±0  1 424 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 2fe265d. ± Comparison against base commit 6cee138.

♻️ This comment has been updated with latest results.

@dixonjoel dixonjoel requested a review from bkeryan August 21, 2025 15:00
@dixonjoel dixonjoel merged commit 128db47 into main Aug 21, 2025
42 checks passed
@dixonjoel dixonjoel deleted the users/jdixon/add-pinmap-v1-proto-package branch August 21, 2025 17:01
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.

4 participants