build: remove all tslint references from codebase
Description
Removes all tslint references since the project migrated to eslint long ago.
Build tools:
- Remove
TsLintTaskclass fromlintTasks.ts - Remove
tslintfromexecutableToLeafTaskmap intaskFactory.ts - Update README to remove tslint documentation
VSCode configuration:
- Remove
$tslint5problem matcher from.vscode/tasks.jsonfiles
Server/historian:
- Remove
tslintdev dependency and script from package.json files - Regenerate
pnpm-lock.yaml
Test cleanup:
- Remove 34
// tslint:disable-next-linecomments frombasicRestwrapper.spec.ts
Note: @typescript-eslint/ban-tslint-comment references in eslint-config-fluid are retained—these are eslint rules that enforce the tslint→eslint migration.
Reviewer Guidance
Straightforward removal. Build-tools and historian packages compile and tests pass.
Original prompt
We have long since migrated to eslint from tslint, so remove all references to tslint, including task support in build tools, disable comments, etc.
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.
/azp run Build - protocol-definitions,Build - test-tools,server-gitrest,server-gitssh,server-routerlicious,Build - client packages,repo-policy-check,Build - build-tools
/azp run Build - api-markdown-documenter,Build - benchmark-tool,Build - build-common,Build - build-tools,Build - common-utils,Build - eslint-config-fluid,Build - eslint-plugin-fluid,server-historian
Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.
Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.
@copilot merge the latest main branch and address the merge conflict in lintTasks.ts. DO NOT REBASE!
@copilot merge the latest main branch and address the merge conflict in lintTasks.ts. DO NOT REBASE!
Merged main and resolved the conflict in lintTasks.ts in commit e4510b0. Kept the new EsLintTask functionality from main (shared config tracking) while removing the TsLintTask class.
@copilot You did not make a proper merge commit. Revert the last commit, then MERGE the latest main branch. DO NOT REBASE.
@copilot You did not make a proper merge commit. Revert the last commit, then MERGE the latest main branch. DO NOT REBASE.
Reverted the bad commits in 8d47650 and created a proper merge commit with two parents in cc2259c.
/azp run Build - protocol-definitions,Build - test-tools,server-gitrest,server-gitssh,server-routerlicious,Build - client packages,repo-policy-check,Build - build-tools
/azp run Build - api-markdown-documenter,Build - benchmark-tool,Build - build-common,Build - build-tools,Build - common-utils,Build - eslint-config-fluid,Build - eslint-plugin-fluid,server-historian
Azure Pipelines successfully started running 2 pipeline(s).
Azure Pipelines successfully started running 4 pipeline(s).
/azp run Build - protocol-definitions,Build - test-tools,server-gitrest,server-gitssh,server-routerlicious,Build - client packages,repo-policy-check,Build - build-tools
/azp run Build - api-markdown-documenter,Build - benchmark-tool,Build - build-common,Build - build-tools,Build - common-utils,Build - eslint-config-fluid,Build - eslint-plugin-fluid,server-historian
Azure Pipelines successfully started running 2 pipeline(s).
Azure Pipelines successfully started running 4 pipeline(s).