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

chore: Update Biome Check workflow and remove unused biome.json file #116

Merged
merged 1 commit into from
Jul 28, 2024

Conversation

ttizze
Copy link
Owner

@ttizze ttizze commented Jul 28, 2024

Summary by CodeRabbit

### リリースノート

- Refactor: Biome Checkワークフローの名前を変更し、セットアップ手順を簡略化しました。
- Chore: 不要な`biome.json`ファイルを削除しました。

Copy link

github-actions bot commented Jul 28, 2024

Image description CodeRabbit

### ウォークスルー
この変更では、Biome Checkワークフローの名前とジョブ名を簡略化し、不要なファイルや手順を削除しました。これにより、セットアップがシンプルになり、メンテナンスが容易になります。

### 変更点
| ファイル | 要約 |
| --- | --- |
| .github/workflows/biome.yaml | Biome Checkワークフローの名前変更、不要な`biome.json`ファイルの削除、ジョブ名の変更、Node.jsバージョン指定とBunインストールの削除、Biomeのセットアップと実行のみを行うように修正 |

Uplevel your code reviews with CodeRabbit Pro

CodeRabbit Pro

If you like this project, please support us by purchasing the Pro version. The Pro version has advanced context, superior noise reduction and several proprietary improvements compared to the open source version. Moreover, CodeRabbit Pro is free for open source projects.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Image description CodeRabbit

Commits Files that changed from the base of the PR and between 05d3651 and 7bcdd18 commits.
Files selected (1)
  • .github/workflows/biome.yaml (1)
Review comments generated (3)
  • Review: 2
  • LGTM: 1

Tips

Chat with Image description CodeRabbit Bot (@coderabbitai)

  • Reply on review comments left by this bot to ask follow-up questions. A review comment is a comment on a diff or a file.
  • Invite the bot into a review comment chain by tagging @coderabbitai in a reply.

Code suggestions

  • The bot may make code suggestions, but please review them carefully before committing since the line number ranges may be misaligned.
  • You can edit the comment made by the bot and manually tweak the suggestion if it is slightly off.

Pausing incremental reviews

  • Add @coderabbitai: ignore anywhere in the PR description to pause further reviews from the bot.

Comment on lines 3 to +6
on:
pull_request:
push:
paths:
- .github/workflows/biome.yaml
pull_request:
workflow_dispatch:

Choose a reason for hiding this comment

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

Image description CodeRabbit

onセクションのインデントが不揃いです。YAMLファイルではインデントが重要なので、修正してください。

   push:
  pull_request:
  workflow_dispatch:
+  push:
+  pull_request:
+  workflow_dispatch:

Comment on lines 1 to 19
name: Biome Check

on:
pull_request:
push:
paths:
- .github/workflows/biome.yaml
pull_request:
workflow_dispatch:

jobs:
biome-check-for-web:
biome-check:
runs-on: ubuntu-latest
permissions:
contents: read
steps:
- uses: actions/checkout@v4
- uses: actions/setup-node@v3
- name: Checkout
uses: actions/checkout@v4
- name: Setup Biome
uses: biomejs/setup-biome@v2
with:
node-version: "20"
- name: Install Bun
run: npm install -g bun
- name: Install dependencies
run: cd web && bun i
- name: Run Biome check
run: |
cd web
bunx @biomejs/biome check . No newline at end of file
version: latest
- name: Run Biome
run: biome ci .

Choose a reason for hiding this comment

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

Image description CodeRabbit

Note: This review was outside of the patch, so it was mapped to the patch with the greatest overlap. Original lines [8-20]

ジョブ名とステップのインデントが不揃いです。また、biome-checkジョブ内のステップもインデントを修正する必要があります。

  biome-check:
     runs-on: ubuntu-latest
     steps:
      - name: Checkout
        uses: actions/checkout@v4
      - name: Setup Biome
        uses: biomejs/setup-biome@v2
         with:
\ No newline at end of file
          version: latest
      - name: Run Biome
        run: biome ci .
+ jobs:
+   biome-check:
+     runs-on: ubuntu-latest
+     steps:
+       - name: Checkout
+         uses: actions/checkout@v4
+       - name: Setup Biome
+         uses: biomejs/setup-biome@v2
+         with:
+           version: latest
+       - name: Run Biome
+         run: biome ci .

@ttizze ttizze merged commit cf8d8f4 into main Jul 28, 2024
5 checks passed
@ttizze ttizze deleted the fix/clbiome branch July 28, 2024 04:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant