-
Notifications
You must be signed in to change notification settings - Fork 382
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
Drop unused properties from device.json #1998
Conversation
@balena-ci retest |
Aren't we using the device instructions property here? |
Good find @klutchell I wasn't able to find any instances of For example this device, https://www.balena.io/docs/learn/getting-started/nitrogen8mm-dwe/nodejs/#provision-device |
I can bring back the instructions for devices that don't have partials to avoid duplication of content. |
Namely, insturctions, machine and state were properties aren't being used anywhere in the documentation and hence were removed. Change-type: patch Signed-off-by: Vipul Gupta (@vipulgupta2048) <[email protected]>
58f26c8
to
30abc2b
Compare
@balena-ci rebase |
30abc2b
to
76ff209
Compare
Check it out @klutchell |
Just out of curiosity I decided to do a bit of "digital archaeology" 😉 and it looks like the EDIT: I wonder if the EDIT2: Nope, my mistake, sorry 🤣 It is still used:
(although of course I have no idea if those pages themselves are also still used!) |
@mehalter I got the impression from our last arch brainstorm that all of this information is migrating to a new device-type source of truth? Will that be a contract type we could leverage here and remove these files? @vipulgupta2048 This PR looks good to me, we can merge and maybe open a new issue to replace this bit entirely? |
True that @lurch the arch property is only being used in one place indeed within a balena push out specifically. Would be good to find a solution for that and clean that as well. @klutchell Indeed in our conversation with @mehalter my plan is to fully wipe out both We can't automate them just yet though because we also need to complete work on screenshots automation for the getting-started guide. With both tasks done + the renovation of getting started guides being done in #2060 we will be in great shape for fully automating the new and improved getting started for the years to come. Check out draft vision: https://docs.google.com/document/d/1oHzkmH_kC6-Z0m7ju5szKDEqE9d_MjhnfY83I2iHgvg/edit# |
Namely, insturctions, machine and state were properties aren't being used anywhere
in the documentation and hence were removed.
Change-type: patch
Signed-off-by: Vipul Gupta (@vipulgupta2048) [email protected]