-
Notifications
You must be signed in to change notification settings - Fork 1
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
Update WPILib Preferences #133
Comments
I think it should be updated. Not sure if it is strictly needed but would be good to do |
@Vultimate1 should be pretty straightforward, copy the corresponding JSON file mentioned above from the starter project in place of the current one and create a pull request to testing |
The JSON file in the starter project is the same as the current file. |
Hmmm that doesn’t seem right, did you update your local WPILib Plugins
installation?
…On Mon, Mar 18, 2019 at 21:25 SKrishnamoorthy ***@***.***> wrote:
The JSON file in the starter project is the same as the current file.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#133 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AEUIR0tuU3d_fULDlP2mHK-9n2zEP1YBks5vYDx_gaJpZM4b2pfs>
.
|
This is what I see in the latest update, can you fix and push?
|
I opened the wpilib_preferences.json file from GitHub and I saw this: |
Which branch were you looking at when you saw the code with "projectYear": "Beta2019"? |
pretty sure it's the same in all of them |
https://github.com/FRCteam4909/2019-Deep-Space/blob/ffb3913942e681edf8d4e876ea4639253799f2d5/.wpilib/wpilib_preferences.json#L4
@TechplexEngineer thoughts on this? I don't think we've had any issues with this and this configuration is never used to the best of my knowledge. We have had this conversation before, but I'd just like to clarify with you.
The text was updated successfully, but these errors were encountered: