Summary:
External version of D724.
Signed-off-by: Yongmin Hong <revi@omglol.email>
Test Plan: Run `pnpm install` and verify installations are successful.
Reviewers: O1 revi & automations, revi
Reviewed By: O1 revi & automations, revi
Differential Revision: https://issuetracker.revi.xyz/D725
Summary:
/shrugs
Signed-off-by: Yongmin Hong <revi@omglol.email>
Test Plan: land and push tag again
Reviewers: O1 revi & automations, revi
Reviewed By: O1 revi & automations, revi
Differential Revision: https://issuetracker.revi.xyz/D634
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
Summary:
Fixes T180. Quick hack, I guess. Following ../eslint-config by using deps not devDeps.
patch version bump for not adding/removing pkgs per se.
Signed-off-by: Yongmin Hong <revi@omglol.email>
Test Plan: land, update prettier pkg and do prettier commands.
Reviewers: O1 revi & automations, revi
Reviewed By: O1 revi & automations, revi
Tags: #npm
Maniphest Tasks: T180
Differential Revision: https://issuetracker.revi.xyz/D480
Summary:
Add xml linter, and bump version to 0.1.0 with changelog rules.
(and types w/ version bump & `.node_version`)
Signed-off-by: Yongmin Hong <revi@omglol.email>
Test Plan: not really testable?
Reviewers: O1 revi & automations, revi
Reviewed By: O1 revi & automations, revi
Differential Revision: https://issuetracker.revi.xyz/D473
Summary:
ESlint flat config blog says [ref] to use dependencies not
peerDependencies (for plugins).
Also add tsc for types as some tools complain about it.
Ref: https://eslint.org/blog/2023/10/flat-config-rollout-plans/
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/D435
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
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
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
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
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
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
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