Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump vue-tsc from 0.39.5 to 1.6.4 #3927

Merged
merged 10 commits into from
Jul 11, 2023
Merged

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github May 4, 2023

Bumps vue-tsc from 0.39.5 to 1.6.4.

Changelog

Sourced from vue-tsc's changelog.

1.6.4 (2023/5/4)

  • feat: support color pickers for inline css
  • feat: support lang="vue" (#3105) - thanks @​HunYan-io
  • fix: when the source type of v-for is any, the item type should be any (#3108) (#3112)
  • fix: avoid slots report error when noPropertyAccessFromIndexSignature enabled (#3106)
  • fix(component-type-helpers): import module causes tsc error (#3114)
  • fix: empty slot name auto-complete not working
  • fix: accessing an unknown slot name does not report an error
  • fix: format adds spaces to v-for if nested template element assigns type to slot props (#2750) (#2757)
  • fix: parentheses auto insert not working in template

Breaking changes

  • drop support for Vue 3.3 slots short definition (#3116)

... (truncated)

Commits

Dependabot compatibility score

You can trigger a rebase of this PR by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
> **Note** > Automatic rebases have been disabled on this pull request as it has been open for over 30 days.

Bumps [vue-tsc](https://github.com/vuejs/language-tools/tree/HEAD/packages/vue-tsc) from 0.39.5 to 1.6.4.
- [Release notes](https://github.com/vuejs/language-tools/releases)
- [Changelog](https://github.com/vuejs/language-tools/blob/master/CHANGELOG.md)
- [Commits](https://github.com/vuejs/language-tools/commits/v1.6.4/packages/vue-tsc)

---
updated-dependencies:
- dependency-name: vue-tsc
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code labels May 4, 2023
@codecov
Copy link

codecov bot commented May 4, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (c7d4244) 86.34% compared to head (cbabbd4) 86.34%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3927   +/-   ##
=======================================
  Coverage   86.34%   86.34%           
=======================================
  Files          66       66           
  Lines        4709     4709           
  Branches      563      563           
=======================================
  Hits         4066     4066           
  Misses        637      637           
  Partials        6        6           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@ras0q ras0q self-assigned this May 7, 2023
@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github May 15, 2023

A newer version of vue-tsc exists, but since this PR has been edited by someone other than Dependabot I haven't updated it. You'll get a PR for the updated version as normal once this PR is merged.

@@ -50,4 +44,12 @@ const comp = computed(() => {
console.error('Unexpected event:', invalid)
return undefined
})

const event = computed(() => {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FIXME: ここをeventAttrsなど別の名前にすると落ちる

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

見てみたんですけど、v-bind使ったときに型がバグってるか複雑なことしないといけなさそうな気がするので、諦めて↓のようにするかeventのままにしておくのがいい気がします...

いい感じに場合分けするのを諦める場合↓

  <sidebar-event-topic-changed
    v-if="event.type === ChannelEventTypeEnum.TopicChanged"
    :datetime="event.datetime"
    :details="event.detail"
  />
  <sidebar-event-subscribers-changed
    v-else-if="event.type === ChannelEventTypeEnum.SubscribersChanged"
    :datetime="event.datetime"
    :details="event.detail"
  />
  <sidebar-event-pinned-changed
    v-else-if="
      event.type === ChannelEventTypeEnum.PinAdded ||
      event.type === ChannelEventTypeEnum.PinRemoved
    "
    :datetime="event.datetime"
    :details="event.detail"
    :type="event.type"
  />
  <sidebar-event-name-changed
    v-else-if="event.type === ChannelEventTypeEnum.NameChanged"
    :datetime="event.datetime"
    :details="event.detail"
  />
  <sidebar-event-parent-changed
    v-else-if="event.type === ChannelEventTypeEnum.ParentChanged"
    :datetime="event.datetime"
    :details="event.detail"
  />
  <sidebar-event-archive-changed
    v-else-if="event.type === ChannelEventTypeEnum.VisibilityChanged"
    :datetime="event.datetime"
    :details="event.detail"
  />
  <sidebar-event-forced-notification-changed
    v-else-if="event.type === ChannelEventTypeEnum.ForcedNotificationChanged"
    :datetime="event.datetime"
    :details="event.detail"
  />
  <sidebar-event-child-created
    v-else-if="event.type === ChannelEventTypeEnum.ChildCreated"
    :datetime="event.datetime"
    :details="event.detail"
  />

Copy link
Member

@ras0q ras0q Jul 10, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PinChangedをPinAddedとPinRemovedに分けてtypeに依存しないようにしたら行けると思うんですがそれだとどうですか?

やってみたら無理だったので諦めます

Comment on lines -78 to +102
// ここでpathを束縛することでcomputed内で戻り値の関数がpathに依存していることが伝わる?
const getComponent = (path: string) =>
defineAsyncComponent(() =>
// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
modalModules[`/src/components/Modal/${path}.vue`]!()
)

const component = computed(() =>
currentState.value
? getComponent(modalComponentMap[currentState.value.type])
: undefined
)
const component = computed(() => {
switch (currentState.value?.type) {
case 'user':
return UserModal
case 'notification':
return NotificationModal
case 'tag':
return TagModal
case 'group':
return GroupModal
case 'channel-create':
return ChannelCreateModal
case 'file':
return FileModal
case 'qrcode':
return QRCodeModal
case 'clip-create':
return ClipCreateModal
case 'clip-folder-create':
return ClipFolderCreateModal
case 'channel-manage':
return ChannelManageModal
case 'group-create':
return GroupCreateModal
case 'group-member-edit':
return GroupMemberEditModal
case 'group-admin-add':
return GroupAdminAddModal
case 'group-member-add':
return GroupMemberAddModal
}
// eslint-disable-next-line no-console
console.error('Unexpected modal type:', currentState.value)
return undefined
})
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

型を参照するために静的にimportしたけどパフォーマンス悪くなる?(未検証)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

手元でビルドすると感じなかった

@ras0q
Copy link
Member

ras0q commented Jul 10, 2023

@mehm8128 諦めの修正しました

Copy link
Contributor

@mehm8128 mehm8128 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございます
動いてるので多分よさそうです

Copy link
Contributor

@nokhnaton nokhnaton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

よさそうです

@ras0q ras0q merged commit 5f8edcf into master Jul 11, 2023
@ras0q ras0q deleted the dependabot/npm_and_yarn/vue-tsc-1.6.4 branch July 11, 2023 12:57
@mehm8128 mehm8128 linked an issue Jul 13, 2023 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

vueのコンポーネントにaria属性を付けられない問題
3 participants