-
Notifications
You must be signed in to change notification settings - Fork 261
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
Document vendordep gradle task #755
Conversation
@Daltz333 , can you mark this as ready for review and |
Can you merge master? |
Won't merge add the 40 commits to this PR? If so, was there a problem with the rebase? |
Anyway, this is waiting on wpilibsuite/GradleRIO#418 so it isn't going anywhere and will probably need to be updated then. Can you mark this as |
Could you merge or rebase onto master? |
@ThadHouse can you make sure I haven't made any technical mistakes in the docs? |
I haven't noticed that the whole article had been reworked. Maybe the section I'm adding should be added somewhere/somehow else in the article? |
The article hasn't been reworked, just moved around. I'm fine with merging this mostly as is (see my comment), as I can just incorporate the information into a revision later. My goal is to better integrate using CAN devices into the zero-to-robot as that's what the overwhelming majority of teams use as their main motor controllers. |
Co-authored-by: Dalton Smith <daltzsmith@gmail.com>
Co-authored-by: Dalton Smith <daltzsmith@gmail.com>
Accompanies wpilibsuite/GradleRIO#418.
Draft until the above commit is merged.