Skip to content

Conversation

@silverwind
Copy link
Member

@silverwind silverwind commented Dec 12, 2025

Add and enable tsgo to lint non-vue files. tsgo is not aware of exports from .vue files, but these are rare so we declare them in globals.d.ts. The primary use case is running pnpm exec tsgo on the CLI to quickly validate non-vue changes (tsgo is about 6 times faster).

This change also enables VSCode typescript.experimental.useTsgo to work without errors in non-vue files.

make lint-js will lint non-vue files twice currently and this can not be avoided until vuejs/language-tools#5381 is resolved, but it's acceptable imho.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Dec 12, 2025
@silverwind silverwind added the skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. label Dec 12, 2025
@silverwind silverwind changed the title Make tsc / tsgo output clean Add tsgo to lint non-vue files Dec 12, 2025
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Dec 12, 2025
@wxiaoguang
Copy link
Contributor

make lint-js will lint non-vue files twice currently and this can not be avoided until vuejs/language-tools#5381 is resolved, but it's acceptable imho.

If the existing lint covers all the cases oftsgo , then I don't see why we should introduce tsgo now which adds unnecessary lint time. When vuejs/language-tools/issues/5381 is resolved, then replace completely at that time.

@silverwind
Copy link
Member Author

silverwind commented Dec 13, 2025

That issue likely won't be resolved anytime soon, at least not cleanly. We can remove the tsgo dependency if you like, it does not affect me, I can still run tsgo even without it being a dependency.

@silverwind silverwind changed the title Add tsgo to lint non-vue files Fix issues related to tsgo Dec 13, 2025
@silverwind
Copy link
Member Author

removed tsgo, now this is just a refactor.

@silverwind silverwind changed the title Fix issues related to tsgo Refactor FileTreeItem Dec 13, 2025
@silverwind silverwind changed the title Refactor FileTreeItem Refactor FileTreeItem type Dec 13, 2025
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Dec 13, 2025
@wxiaoguang wxiaoguang enabled auto-merge (squash) December 13, 2025 13:03
@wxiaoguang wxiaoguang merged commit ac8308b into go-gitea:main Dec 13, 2025
23 checks passed
@GiteaBot GiteaBot added this to the 1.26.0 milestone Dec 13, 2025
@silverwind silverwind deleted the item branch December 13, 2025 13:08
zjjhot added a commit to zjjhot/gitea that referenced this pull request Dec 15, 2025
* giteaofficial/main:
  Remove undocumented support of signing key in the repository git configuration file (go-gitea#36143)
  Enable gocheckcompilerdirectives linter (go-gitea#36156)
  Fix code highlighting on blame page (go-gitea#36157)
  Check user visibility when redirecting to a renamed user (go-gitea#36148)
  Fix bug when viewing the commit diff page with non-ANSI files (go-gitea#36149)
  Refactor `FileTreeItem` type (go-gitea#36137)
  Fix various bugs (go-gitea#36139)
  Fix issue close timeline icon (go-gitea#36138)
  Add permissions to`files-changed` jobs (go-gitea#36142)
  Add explicit permissions to all actions workflows (go-gitea#36140)
  Bump `actions/checkout` to v6 (go-gitea#36136)
  Hide RSS icon when viewing a file not under a branch (go-gitea#36135)
  Fix SVG size calulation, only use `style` attribute (go-gitea#36133)
  Add sorting/filtering to admin user search API endpoint (go-gitea#36112)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/frontend skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants