John Whitlock

I think we all agree that additional keys is not the solution, no need for further beating of that straw man. I prefer ``__l10nkey:`` as a prefix because: * It...

See #67 for what the css/properties.json would look like with the prefixes applied everywhere. Some notable things: * It appears a lot of the values are lookup keys. * Values...

Thanks for your feedback. I think some of it would be clearer on #67, where you can add your comments to specific lines of the changed JSON, and I can...

After thinking about it for a while, I'm 👎 on using the prefix, and I like the idea of a different key for prose descriptions more, such as: ```json "align-self":...

I was thinking something like this: 1. mdn/data just has the English strings. 2. The string extractor looks for keys with the ``*Text`` name, or we have an extra config...

@octref this project isn't on the timeline, as far as I'm aware. The browser-compat-data project will continue to be the focus until the data is migrated, hopefully by July. If...

@dependabot rebase

It is interesting that all tests pass, since [connect-redis](https://github.com/tj/connect-redis#api) says a different invocation is required: ``` // redis@v4 const { createClient } = require("redis") let redisClient = createClient({ legacyMode: true...

Force-pushed to fix the command. [`coveralls` ran in CircleCI](https://app.circleci.com/pipelines/github/mozilla/blurts-server/5322/workflows/1a1eded3-39fd-4c9a-943c-133daf442000/jobs/31725?invite=true#step-105-169) and coverage was [uploaded to coveralls.io](https://coveralls.io/builds/53100422). It is skipped for me locally (since I never setup `.coverage.yml`). We're missing a coverage...

Rebased on current main and made changes based on feedback: * Script is now `scripts/is-coveralls-configured.js`, using kabob-naming for consistency with other scripts * `npm test` now calls `scripts/is_coveralls_configured.js` directly, and...