[examples] DigitalCommunication, I2CCommunication: Add tests #4865
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Starlight220
force-pushed
the
dc-test
branch
2 times, most recently
from
e46155b to
aaa36b6
Compare
Starlight220
force-pushed
the
dc-test
branch
from
933f530 to
b56073b
Compare
|
Have you tried creating a project from this with the vscode plugin? Just want to make sure it still compiles/works fine when taken out of the allwpilib context- especially if "enable desktop support" is unchecked |
|
I'm not sure how to get vscode to pull examples from a dev build. |
|
publish the allwpilib build and then build the vs code extension and it will pull in the dev build examples/templates. |
|
the examples test folder doesn't appear to be published |
|
That's semi-intentional, and would need vscode changes. We haven't decided that we want those tests in user projects -- keeping them there would fail deploys if the code is changed. |
Starlight220
changed the title
PeterJohnson
approved these changes
Jan 9, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.