Commit graph

16 commits

Author SHA1 Message Date
ab283848b8
CI(npm): Refer local directory as file path
Summary:
Avoid updating local pkgs.

Also add `prefer-dedupe=true` on `.npmrc`.

Signed-off-by: Yongmin Hong <revi@omglol.email>

Test Plan: `npm ci`.

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D428
2024-06-25 16:09:48 +09:00
f0fb5d1812
npm(eslint-config): add bugs entry
Summary:
Well, it's unlikely someone will need file a bug for this pkg but tools
are referring to GitHub issues which are disabled.

Not bumping the version because trivial.

Signed-off-by: Yongmin Hong <revi@omglol.email>

Test Plan: When new version is published, go to deps.dev and verify issues label is set to `revi.xyz/new`

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D416
2024-06-24 16:58:54 +09:00
09d55a76c5
npm(prettier-config): add .devcontainer/*.json as jsonc.
Summary:
[devcontainer.json](https://containers.dev/implementors/spec/#devcontainerjson)
is a "JSON with Comments" file.

Signed-off-by: Yongmin Hong <revi@omglol.email>

Test Plan: Watch how prettier behaves with rWEBXYZ `.devcontainer/devcontainer.json`.

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D400
2024-06-22 17:38:06 +09:00
bfab2ea75c
CI(npm): pin npm package version
Summary:
Ref T176.

Signed-off-by: Yongmin Hong <revi@omglol.email>

Test Plan: `npm ci`

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Maniphest Tasks: T176

Differential Revision: https://issuetracker.revi.xyz/D393
2024-06-21 16:15:24 +09:00
88f952d4cd
npm(prettier-config): add homepage
Summary:
link to directory. No version bump for this.

Signed-off-by: Yongmin Hong <revi@omglol.email>

Test Plan: verify the homepage when the new release is cut

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D387
2024-06-21 16:09:10 +09:00
a287bd06ca
npm(eslint-config): version bump with deps update
Summary:
add @types, --save-exact for prettier(-config), add homepage.

Signed-off-by: Yongmin Hong <revi@omglol.email>

Test Plan: No visible changes.

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D378
2024-06-20 15:01:11 +09:00
b189fe62aa
npm(prettier-config): add tsconfig to jsonc
Summary:
`tsconfig.json` has comments, so it is jsonc.

Cf. https://aka.ms/tsconfig.json

Also add tsconfig.json in rootdir and update arclint to ignore
`tsconfig.json`.

Signed-off-by: Yongmin Hong <revi@omglol.email>

Test Plan: Try prettier checks on `tsconfig.json` after merging.

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D376
2024-06-20 14:54:14 +09:00
c079c483f8
npm(prettier-config): trailingComma: 'none' for jsonc
Summary:
Seems like prettier wants to put trailingComma to JSONc.

Signed-off-by: Yongmin Hong <revi@omglol.email>

Test Plan: `arc land`, push new release, test.

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D366
2024-06-19 14:32:59 +09:00
cbfa3411b3
CI(GitHub Actions): fix npm-pub.yml to publish correctly
Summary:
Fix P185 failure.

Also fix L1 (`npm pkg fix`) for (eslint|prettier)-config.

Signed-off-by: Yongmin Hong <revi@omglol.email>

Test Plan: `arc lint`, and push tag again

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D358
2024-06-18 19:26:34 +09:00
5c16c87f8d
npm(prettier-config): add prettier plugins, add rules, consolidate
Summary:
And run `npm run p:w .` for that, and add .prettierignore so `sh` plugin ignores them.

Signed-off-by: Yongmin Hong <revi@omglol.email>

Test Plan: `npm run p:c .` and `npm publish`.

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D355
2024-06-18 18:46:30 +09:00
959705258e
npm(eslint-config): correct wrong peerDependency
Summary:
eslint-config-prettier should have been peerDependency not devDependency.
Correct them by moving it.

(Also do some yml cleanups.)

Signed-off-by: Yongmin Hong <revi@omglol.email>

Test Plan: `npm ci`.

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D342
2024-06-17 12:53:55 +09:00
9afee876a5
npm(README): add shields.io badges
Summary:
Not worthy of new release though.

Signed-off-by: Yongmin Hong <revi@omglol.email>

Test Plan: Browse the markdown in editor's preview mode

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D337
2024-06-17 12:46:50 +09:00
0f4b7c97e3
CI: add eslint
Summary:
Signed-off-by: Yongmin Hong <revi@omglol.email>

npm: use @revi/eslint-config

Also change eslint format, and publish `v0.0.2`.
Also prepare for eslint-config-wikimedia when they have eslint9 support.

Signed-off-by: Yongmin Hong <revi@omglol.email>

Test Plan: `npm run eslint`.

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D332
2024-06-16 15:50:44 +09:00
be3db3c067
npm(eslint-config): eslint shard config
Summary:
Yet another shared config for consolidation.

Signed-off-by: Yongmin Hong <revi@omglol.email>

Test Plan: `arc land`, `npm publish`, and install it.

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D331
2024-06-16 14:49:55 +09:00
41b70cb468
npm(prettier-config): add README/author data, make prettier peerDependency
Summary:
Bump to 0.0.2.

For easier browsing...

Signed-off-by: Yongmin Hong <revi@omglol.email>

Test Plan: `arc land`, and `npm publish`

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D310
2024-06-14 19:49:11 +09:00
8865e172f9
npm(prettier-config): add new sharable config
Summary:
Docs: https://prettier.io/docs/en/configuration#sharing-configurations

To be published @ npm and imported elsewhere.

Signed-off-by: Yongmin Hong <revi@omglol.email>

Test Plan: no test can be performed at this time; needs to be landed, published to npm, and then we can test from other tepo importing this.

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D302
2024-06-14 17:18:03 +09:00