Skip to content

Conversation

@dixonjoel
Copy link
Collaborator

What does this Pull Request accomplish?

The ni.measurementlink.proto package includes the generated code for pin_map_context.proto

Why should this Pull Request be merged?

Need the pin_map_context.proto generated code as a leaf dependency.

What testing has been done?

Will publish a package for testing after submission.

@github-actions
Copy link
Contributor

github-actions bot commented Aug 15, 2025

Test Results

   26 files  ±0     26 suites  ±0   30s ⏱️ -1s
  136 tests ±0    136 ✅ ±0  0 💤 ±0  0 ❌ ±0 
1 344 runs  ±0  1 344 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 9a57899. ± Comparison against base commit fe5644a.

♻️ This comment has been updated with latest results.

@dixonjoel dixonjoel merged commit cdc17f3 into main Aug 15, 2025
33 checks passed
@dixonjoel dixonjoel deleted the users/jdixon/ni-measurementlink-pinmap-package branch August 15, 2025 17:34
name = "ni.measurementlink.proto"
version = "0.1.0-dev0"
license = "MIT"
description = "Protobuf data types for NI gRPC APIs"
Copy link
Collaborator

Choose a reason for hiding this comment

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

update

[tool.poetry.dependencies]
python = "^3.9"
protobuf = {version=">=4.21"}
nitypes = {version=">=0.1.0dev3", allow-prereleases=true}
Copy link
Collaborator

Choose a reason for hiding this comment

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

delete

Comment on lines +52 to +53
# Use a prerelease version of hightime for testing.
hightime = { version = ">=0.3.0.dev0", allow-prereleases = true }
Copy link
Collaborator

Choose a reason for hiding this comment

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

delete

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.

5 participants