Skip to content

fix: apply noUncheckedIndexedAccess true - #57

Open
tsukuha wants to merge 4 commits into
gitFoxCode:mainfrom
tsukuha:tsukuha/apply-noUncheckedIndexedAccess
Open

fix: apply noUncheckedIndexedAccess true#57
tsukuha wants to merge 4 commits into
gitFoxCode:mainfrom
tsukuha:tsukuha/apply-noUncheckedIndexedAccess

Conversation

@tsukuha

Copy link
Copy Markdown

thanks for looking my PR about #55

Excuse me, about created to fix about this issue without waiting for your action.

about issue

close: #55

test logs

スクリーンショット 2024-06-13 12 10 08

@tsukuhatsukuha changed the title #55 apply noUncheckedIndexedAccess true#55 fix: apply noUncheckedIndexedAccess trueJun 13, 2024
@tsukuhatsukuha changed the title #55 fix: apply noUncheckedIndexedAccess truefix: apply noUncheckedIndexedAccess trueJun 13, 2024
Comment threadsrc/runtime/components/nuxt-icon.vue Outdated
)
return
}
const rawIcon = rawIconImport()

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

i think the await goes here

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

@Miles0908
thanks for your review!!

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

vue-tsc error at line 27 of nuxt-icon.vue, while activated noUncheckedIndexedAccess in tsconfig

2 participants

@tsukuha@Miles0908