This resolves #393 but perhaps a followup should be created to reduce the frequency of github API calls.
The promise chain is partially moved into a function so that the response variable is available. When an error occurs, the counts are replaced with "---". Not sure what the best thing to show here is, perhaps 'ERR" instead? I don't like 'ERR' because it is not really an error, just limits. I don't think it is worth a more prominent error display, as for the vast majority of displays of the repo, people don't really care about the HELP WANTED type counts. But the display of ">99" as at present is confusing and incorrect.
This resolves #393 but perhaps a followup should be created to reduce the frequency of github API calls.
The promise chain is partially moved into a function so that the
response
variable is available. When an error occurs, the counts are replaced with "---". Not sure what the best thing to show here is, perhaps 'ERR" instead? I don't like 'ERR' because it is not really an error, just limits. I don't think it is worth a more prominent error display, as for the vast majority of displays of the repo, people don't really care about the HELP WANTED type counts. But the display of ">99" as at present is confusing and incorrect.