Skip to content

readme: Add python and rust packages #253

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

Merged
merged 1 commit into from
Mar 1, 2025
Merged

Conversation

amacneil
Copy link
Contributor

@amacneil amacneil commented Feb 27, 2025

  • Update readme with with new python and rust SDKs (with disclaimer about beta status)
  • Remove references to packages not contained in this repo

@amacneil amacneil force-pushed the adrian/readme-py-rust branch from edf590f to 8cbe4d3 Compare February 28, 2025 23:34
Copy link
Member

@jtbandes jtbandes left a comment

Choose a reason for hiding this comment

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

I forget how we ended up with a HTML table here but wonder if it would be more manageable if using MD table syntax?

README.md Outdated
Comment on lines 44 to 62
<td>Foxglove SDK for C</td>
</tr>
<tr>
<td>

Python implementation of the WebSocket protocol<br/>Repo: [foxglove/ws-protocol](https://github.com/foxglove/ws-protocol/tree/main/python)
[foxglove_cpp](./cpp)

</td>
<td></td>
<td>Foxglove SDK for C++</td>
Copy link
Member

Choose a reason for hiding this comment

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

a bit early to add these...but whatever floats your boat

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah I was going to wait to merge this, but I might just remove them and merge it sooner

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok I removed c/c++ and added an additional disclaimer about beta testing

@amacneil amacneil force-pushed the adrian/readme-py-rust branch 2 times, most recently from ce30158 to 0dea454 Compare March 1, 2025 00:22
@amacneil
Copy link
Contributor Author

amacneil commented Mar 1, 2025

I forget how we ended up with a HTML table here but wonder if it would be more manageable if using MD table syntax?

changed it a little while back because we were getting a huge markdown diff each time, due to prettier wanting to pad out each cell to the same width

@amacneil amacneil changed the title readme: Add new python and rust packages readme: Add python and rust packages Mar 1, 2025
@amacneil amacneil force-pushed the adrian/readme-py-rust branch 7 times, most recently from 8a4981e to d58bc5c Compare March 1, 2025 00:50
@amacneil amacneil force-pushed the adrian/readme-py-rust branch from d58bc5c to 4a1f91a Compare March 1, 2025 01:06
@amacneil amacneil merged commit 7439e0c into main Mar 1, 2025
36 checks passed
@amacneil amacneil deleted the adrian/readme-py-rust branch March 1, 2025 01:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants