Commit graph

14 commits

Author SHA1 Message Date
8672f0b7bc
npm: update (eslint|prettier)-config packages
Summary:
Originally should've migrated to sync via T182 but work is not complete
so for the time being do it manually.

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

Test Plan: Land and test.

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D633
2024-08-01 18:59:12 +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
ba384f8349
meta(.vscode): add directory
Summary:
Add extensions and settings, and gitattributes.

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

Test Plan: land it.

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D367
2024-06-19 14:40:02 +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
b1740bd0cd
CI(.arclint): use Pinterest Arcanist Linter
Summary:
Thanks pinterest!

Also fix some GitHub Actions error.

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

Test Plan: run `arc lint`?

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D325
2024-06-16 14:45:56 +09:00
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
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
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
cdf70a8ee2
container: build and publish containers
Summary:
Cheap docker hub alt :P

Dockerfile from [googlechrome/lighthouse-ci](https://github.com/GoogleChrome/lighthouse-ci/tree/main/docs/recipes/docker-server)

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

Test Plan: Check GitHub Actions runs?

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D132
2024-05-23 16:12:11 +09:00
bf5c8a9281
Robots.txt: f off PetalBot
Summary:
Is on .xyz/robots.txt blacklist. add it here as well.

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

Test Plan: Deploy

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D125
2024-05-23 04:20:04 +09:00
08863ae041 time.php: add DOCTYPE
Summary:
`<!DOCTYPE html>`, a good thing to do ™️.

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

Test Plan: load the page in browser.

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D82
2024-05-13 20:02:48 +09:00
d75d8cf807
.arclint: more rules, exclusions
Summary:
Probably standard changes (tm) at this point.

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

Test Plan: `arc lint`

Reviewers: O1 revi & automations, revi

Reviewed By: O1 revi & automations, revi

Differential Revision: https://issuetracker.revi.xyz/D79
2024-05-12 19:02:50 +09:00
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
d2f7806542
meta: Add .arcconfig
Summary: Either Phab differential or Gerrit (to be installed) but not sure what to use.

Test Plan: `arc patch`, verify lint works and connects to this instance.

Reviewers: #acl_sudoers, #blessed_reviewers, revi

Reviewed By: #acl_sudoers, #blessed_reviewers, revi

Differential Revision: https://issuetracker.revi.xyz/D7
2024-05-02 18:28:21 +09:00