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.

Sign up for GitHub

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

Jump to bottom

chore: Airtableからルール出力する運用をやめる #568

Merged

ibulog merged 2 commits into main from quit-airtable

Oct 28, 2024

Contributor

ibulog commented Oct 25, 2024 •

edited

Loading

課題・背景

SmartHR Design Systemの用字用語ページについて、Airtableから生成する運用からmdxに直接書く運用に変更された
よって、用字用語が下記の3重に管理されている状態
- SmartHR Design Systemのmdx
- Airtable
- textlintのルール
3重管理を解消するため、Airtableからtextlintルールを生成する運用を廃止し、textlintルールもymlファイルを直接編集する運用に変更する

やったこと

Airtableからtextlintルールを出力するためのコードを削除

やらなかったこと

申し送り

GitHubリポジトリに登録されたCredentialsは後ほど削除します

動作確認

ルール自体の変更はないので、差分の確認のみお願いします！

正しいと判定される想定の文章

誤りと判定される想定の文章


          chore: Airtableからルール出力する運用をやめる

14eb34e

Contributor

diescake commented Oct 25, 2024

@ibulog .env.example もAirtable用の情報しかないので消せそうです！

diescake reviewed

View reviewed changes

package.json Outdated Show resolved Hide resolved


          chore: 環境変数は読み込む必要がなくなる

6d96a78

ibulog requested a review from diescake

October 28, 2024 01:15

diescake approved these changes

View reviewed changes

Contributor

diescake left a comment

あざます！GoGo〜〜！

ibulog merged commit 2889552 into main

2 checks passed

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

Labels

None yet