0xSpaceShard / 0xSpaceShard/starknet-hardhat-example

Improve checks in CI/CD

オープン
#116 コメント 0 件 リアクション 0 件 担当者 1 名 @Nathan-SL が担当を希望しています GitHub で見る
enhancement
主要言語
TypeScript
スター
88
フォーク
29
PR マージ指標
30日以内にマージされた PR はありません

説明

Currently we just lint, but we would benefit from doing what I have in my pre-commit script:
```bash
set -e

# check typescript correctness
npx tsc --noEmit

# check all .sh files are executable
invalid_scripts=0
for script in $( find . -type f -iname *.sh ); do
if [ ! -x $script ]; then
echo "Script $script is not executable"
invalid_scripts=$((invalid_scripts + 1))
fi
done
if [ $invalid_scripts != 0 ]; then
exit 1
fi

# check there are no it.only occurrences in the tests
grep -re "it\.only(" $(git ls-files '*.ts') && echo "Error: There are files with it.only(...)" && exit 2

# lint
npm run lint
```

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

Look for the CI/CD configuration file (likely .github/workflows/*.yml) to understand the current linting step. The issue provides a bash script to integrate: it runs TypeScript type checking, ensures .sh files are executable, checks for stray it.only in test files, and then runs linting. Start by locating where the CI script runs and add these checks in the appropriate order. Verify by running the script locally and then updating the CI configuration.

索引モデルが issue の本文から書いたものです。

評価

技術スタック
bash, github-actions, nodejs, shell, typescript
領域
ci-cd, testing, tooling
issue の種類
機能追加
難易度
2/5
見積もり時間
1〜3時間
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
40/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。