* chore: fix ci & tests
* Revert "chore: update to es module (#1869)"
This reverts commit db9a401c15.
* Revert "chore: fix ci & tests (#1870)"
This reverts commit ecb88ce202.
* add grandfathered list of apps with non-ssl links
* add test to ensure app links use ssl
* document the requirement that links must use ssl
* remove unneeded linter pragma
* chore: switch links from http to https if possible
Switch made by https://gist.github.com/ckerr/41f8e7c8add833074c8142ed4858da99
* test: add contribution guidelines tests for app descriptions
* chore: grandfather in existing apps
There are literally hundreds of descriptions not following the guidelines,
so for now let's just enforce them on new changes.
* fix recent regressions
these regressions were introduced since this PR was created.
* fix npm audit warnings by bumping dependencies.
Note: this PR doesn't make the leap of switching from `GitHub`
to `octokit/rest` Instead, it takes the easier path of bumping
the `GitHub` dependency to the final release before the rename.
* bump check-for-leaks dependency
* update github dependency to @octokit/rest
* refactor: use json-stable-stringify in colors.json
This ensures that the generated files will only have diffs when
the contents change, rather than due to arbitrary sorting of an
object's properties
* refactor: sort require calls
* refactor: simplify color object creation
* feat: if we already have color info for an app, reuse it
* feat: only recalculate colors when icons change
Add a revHash for the icon file & use the old color values
if the revHash is unchanged since the last run.
* refactor: skip redundant file loads
Previously we loaded once to get the revHash, then a second time to
compute the complimentary colors. Now we load it once into a buffer,
then use the buffer both tasks
* chore: update package-lock.json
since this branch adds three new dependencies
* chore: update colors file
this update adds the icon file revHashes
* refactor: control colors endpoints for testability
* chore: fix linting
* test: add tests for color script
* docs: add some function comments
* refactor: color script readability
* chore: update package-lock
* chore: update package-lock
* refactor: better color error.log testing
* fix: incorrect terminology in comments
* fix: indentation whitespace
* fix: silence incorrect linter warning
* fix: hide intentional test errors
The tests feed bad data into the color reader, so it logs errors.
Dumping these errors during `npm run test` makes the output noisy.
So instead of having an `console.error` spy, actually intercept it
and test to confirm the errors the expected information.
* chore: remove sinon, sinon-chai dependencies
Sinon was unused after the `console.error` spy was removed.
* fix: function name typo
* fix: remove unused variable
* initial swing at categories
* Finished first round of categories
* Added tests for category
Also, documented category in contributing and added it to the wizard.
Also fixed broken categories on some apps.
* Updated unreachable sites
* Fix broken apps
* Fix standardjs error