revi/sandbox
1
0
Fork 0
Code Issues Wiki Activity
76 commits 15 branches 25 tags 58 MiB
Commit graph

7 commits

Author SHA1 Message Date
Yongmin Hong
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
Yongmin Hong
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
Yongmin Hong
01e6487932
npm: use shared prettier config 
Summary:
Shared config testbed, yeah.

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

Test Plan: `npm run p:c .` should run as it is.

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D305
 2024-06-14 17:24:13 +09:00
Yongmin Hong
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
Yongmin Hong
04798b45fd
style: npm run p:w . 
Summary:
Which is essentially `npx prettier --write .`

Also fix `jshint` (possible) FP, wrong chars.

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

Test Plan: `npm run p:w .` and verify no changes made.

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D217
 2024-06-09 22:11:33 +09:00
Yongmin Hong
aca8c5967b
CI: new package and configs 
Summary:
add following npm packages:

 - jshint
 - prettier

Add `prettier` configs and `.arclint` accordingly.

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

Test Plan: `npm run pcheck`, `npm run pwrite`.

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D216
 2024-06-08 19:04:29 +09:00
Yongmin Hong
d45c16bde1
meta: arclint: add few more rules 
Summary:
I probably want few more rules (ie. csslint)

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

Test Plan: `arc lint`

Reviewers: #acl_sudoers, #blessed_reviewers, revi

Reviewed By: #acl_sudoers, #blessed_reviewers, revi

Differential Revision: https://issuetracker.revi.xyz/D20
 2024-05-04 17:34:01 +09:00