Skip to content

Conversation

@cjlawson02
Copy link
Owner

Fixes #167 #166 #163

Description todo

@cjlawson02 cjlawson02 requested a review from Copilot April 23, 2025 01:27
@cjlawson02 cjlawson02 self-assigned this Apr 23, 2025
Copy link

Copilot AI left a comment

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 adds support for protobuf and structs along with an example robot application and updates several project dependencies. The changes include regenerated documentation for the NetworkTables classes and updates to the search and navigation assets.

Reviewed Changes

Copilot reviewed 91 out of 107 changed files in this pull request and generated no comments.

File Description
docs/classes/NetworkTablesTypeInfos.html Regenerated documentation with updated relative path attributes and script updates.
docs/classes/NetworkTables.html Updated documentation output with minor formatting adjustments.
docs/assets/search.js Updated the search index data string generated by the documentation tooling.
docs/assets/navigation.js Updated the navigation data string reflecting changes from dependency updates.
Files not reviewed (16)
  • .eslintrc.json: Language not supported
  • .vscode/extensions.json: Language not supported
  • .vscode/settings.json: Language not supported
  • apps/example-robot/.wpilib/wpilib_preferences.json: Language not supported
  • apps/example-robot/build.gradle: Language not supported
  • apps/example-robot/gradle/wrapper/gradle-wrapper.properties: Language not supported
  • apps/example-robot/gradlew: Language not supported
  • apps/example-robot/gradlew.bat: Language not supported
  • apps/example-robot/project.json: Language not supported
  • apps/example-robot/settings.gradle: Language not supported
  • apps/example-robot/vendordeps/WPILibNewCommands.json: Language not supported
  • docs/assets/main.js: Language not supported
  • docs/assets/hierarchy.js: Language not supported
  • apps/example-robot/src/main/java/frc/robot/Robot.java: Language not supported
  • apps/example-robot/src/main/java/frc/robot/Main.java: Language not supported
  • apps/example-client/src/main.ts: Language not supported

@codecov
Copy link

codecov bot commented Apr 23, 2025

Codecov Report

Attention: Patch coverage is 99.23077% with 1 line in your changes missing coverage. Please review.

Project coverage is 90.02%. Comparing base (489d241) to head (f73e0b3).

Files with missing lines Patch % Lines
packages/ntcore-ts-client/src/lib/socket/socket.ts 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #168      +/-   ##
==========================================
+ Coverage   89.74%   90.02%   +0.28%     
==========================================
  Files          15       15              
  Lines        1131     1163      +32     
  Branches      232      224       -8     
==========================================
+ Hits         1015     1047      +32     
  Misses        112      112              
  Partials        4        4              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@cjlawson02 cjlawson02 force-pushed the structs branch 3 times, most recently from d07cede to 0b39dd0 Compare April 26, 2025 20:39
@cjlawson02 cjlawson02 changed the title feat: add support for protobuf and structs, example robot app, update dependencies feat: add support for protobuf and structs, example robot app Apr 26, 2025
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.

Client barfs when subscribing to .schema

2 participants