Skip to content

Refactor FileTreeItem type#36137

Merged
wxiaoguang merged 10 commits into
go-gitea:mainfrom
silverwind:item
Dec 13, 2025
Merged

Refactor FileTreeItem type#36137
wxiaoguang merged 10 commits into
go-gitea:mainfrom
silverwind:item

Conversation

@silverwind

@silverwind silverwind commented Dec 12, 2025

Copy link
Copy Markdown
Member

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
Comment thread web_src/js/globals.d.ts Outdated
@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
Copy Markdown
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

silverwind commented Dec 13, 2025

Copy link
Copy Markdown
Member Author

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
Copy Markdown
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)
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Mar 13, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. 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