I consider this fixed for now with 0242dfcb0f83e3ca906461b0b3e55f1832d5d528. Wanna make sure I am not jumping the gun on that. When it comes to like what to do with dealing with apps .env files I…
Also, what can be tested here? Can I mock the JSON to do offline tests?
I've seen rumblings on mocking an actual http call in a unit test for a mock one that returns what ever you want. No rush…
Your function never returns this error. It is just logging it and exiting the program. You should change this to return nil, err
. This can then be dealt with by the command itself.
This will never be run because you never return an error, see comment on GetAppsJSON
I am not totally against this approach. I worry about the speed of it but honestly might just say fuck it and use this as our baseline for sorting maps.
Not much to review since most of it is good! Just one note, and a situation where you don't return an error where you should. So gunna label it as requesting a change for that reason but its a one line change 💙