8000 Lint エラーの修正 by aster-mnch · Pull Request #56 · vuejs-jp/learn.nuxt.com · GitHub
[go: up one dir, main page]

Skip to content
This repository was archived by the owner on May 1, 2025. It is now read-only.

Lint エラーの修正 #56

Merged
merged 3 commits into from
Jul 29, 2024
Merged

Lint エラーの修正 #56

merged 3 commits into from
Jul 29, 2024

Conversation

aster-mnch
Copy link
Collaborator

GHA で確認している eslint, textlint エラーを修正しました。

The following textlint errors have been fixed.

```
vuejs-jp/no-mix-dearu-desumasu: 本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である["
Total:
である  : 1
ですます: 13

learn.nuxt.com/content/1.vue/1.index.md:7:43
                                                                                       v
    6.
    7. Nuxt は、ユーザーインターフェイスを構築するためのプログレッシブなフレームワークである[Vue 3](https://vuejs.org/)を使用しています。このセクションでは、Vue の基本について説明します。
    8.
                                                                                       ^
```
@aster-mnch
Copy link
Collaborator Author

リント系とはまた別のところで落ちてた
https://github.com/vuejs-jp/learn.nuxt.com/actions/runs/10141803201/job/28039759819?pr=56

内容的に GHA 内で package instlall するときに ci じゃなくて install を使ってるから、pnpm-lock.yaml の lockfileVersion が更新されてるみたい。
これも対応しますかぁ

@aster-mnch
Copy link
Collaborator Author

もしかして pnpm ci ってない?

@aster-mnch
Copy link
Collaborator Author

pnpm i --frozen-lockfile 使えば良さそう

@ubugeeei
Copy link
Member

あ〜 いや
多分 lockfile ミスってます。
pnpm は ci 環境だと自動で frozen になるはずです。
一旦放置で OK です

@ubugeeei
Copy link
Member
ubugeeei commented Jul 29, 2024

って思ったけど actions ってならないんだっけ... (どちらにせよ一旦無視で OK です)
https://pnpm.io/next/cli/install#--frozen-lockfile

@aster-mnch
Copy link
Collaborator Author

(どちらにせよ一旦無視で OK です)

了解です、とりあえずこの PR はオープンにします 🙆‍♂️
(すでに #57 も作っていますが)

@aster-mnch aster-mnch changed the title WIP Lint エラーの修正 Lint エラーの修正 Jul 29, 2024
@aster-mnch aster-mnch requested a review from ubugeeei July 29, 2024 09:51
@ubugeeei ubugeeei merged commit ceb8843 into main Jul 29, 2024
1 of 2 checks passed
@ubugeeei ubugeeei deleted the fix-lint branch July 29, 2024 09:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0