Skip to content
This repository has been archived by the owner on Feb 4, 2023. It is now read-only.

CIの権限、及び処理を修正, PR templateを修正 #176

Merged
merged 145 commits into from
Sep 18, 2022
Merged

CIの権限、及び処理を修正, PR templateを修正 #176

merged 145 commits into from
Sep 18, 2022

Conversation

re-taro
Copy link
Member

@re-taro re-taro commented Sep 18, 2022

fix: CIの権限、及び処理を修正 (c48cce6)
doc: PR templateを修正 (1d27b43)

renovate bot and others added 30 commits July 9, 2022 11:49
…(non-major)

Update dependency next to v12.2.1
Update dependency eslint-plugin-storybook to v0.6.0
…ndencies

Update dependency jest-extended to v3
…(non-major)

Update dependency next to v12.2.2
style-dictionaryをTailwind CSSに対応させた
Merge pull request #59 from suzuka-kosen-festa/feature/#57
現在は協賛する企業がわからないため空白
協賛企業の情報を載せるファイルを作成した
Merge pull request #68 from suzuka-kosen-festa/feature/#55
re-taro and others added 18 commits August 22, 2022 16:28
* ✨ hero sectionはセクションだがコンテンツモデル的にheaderと同格と判断

section -> header

* ✨ twitterの埋め込みの作成

* 🔧 lintのsecretsになりうる箇所へのエラーを廃止

* ✨ storiesの追加

* ✅ testの追加

* 📸 snapshotの追加

* ✨ snsセクションを実装

Instagramは @re-taro に権限が与えられていなかったため実装不可能

* ✨ storiesの追加

* ✅ testの追加

* 📸 snapshotの追加
* Update: Propertiesの追加

* Create: sponsor-sectionの実装

* Create: テストを追加

* Create: storiesを追加

* Create: snapshotを追加

* Fix: 型用意されてるの使わないとか愚か?

* Update: sponsorsを配列に変更

* Update: 企業一覧をリストで再実装

* Update] snapshotの更新

* Update: 不要な要素の削除

* Update: リストの書き方を変更

* Update: 汚染を破壊

* Update: テストの追加

* Fix: テストの内容を修正

* Update: snapshotの更新

* Update: 整形

* Update: snapshotの更新
* Add: next-pwaを導入

* Update: PWA用の設定を追加

* Update: issueの解決されているversionを使う

* Update: ignoreの更新

* Fix: next-pwaのバージョンを下げた

* Create: manifestを生やした

* Update: Headの中身を戻した

* Remove: manifestを統一

* Update: manifest変更に伴いパスの変更

* Update: snapshotの更新

* Update: manifestをseoに戻した

* Update: manifestのパスを変更

* Update: snapshotの更新

* Update: manifestは読み込まれたので消した

* Update: 重複しているmetaの削除

* Update: snapshotの更新
* ✨ space-yを指定

それに伴ってsnapshotを更新

* Twitterコンポーネントのレスポンシブ化及びFCPの改善

width: 300px以下のスマホへの動作保証はありません。

* ✨ β版のUIの作成

* 👷 chromaticへの登録をstoriesの変更ではなくcomponents自体の変更時に発火させるようにした

storiesの変更時というのはあまり本質的じゃない

* ✨ anchor textを統一した

* ✨ 余分なwidth: screenを消去した

* [skip ci] auto-fixable format by github-actions

* ✨ URLを更新した

* ⚡️ 整形

Co-authored-by: re-taro <re-taro@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* Update devDependencies to v29

* update: add babel dependence

* [skip ci] auto-fixable format by github-actions

* format code

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Rintaro Itokawa <me@re-taro.dev>
Co-authored-by: re-taro <re-taro@users.noreply.github.com>
* Update dependencies (non-major)

* feat: Cloudflareを用いた最適化への対応

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Rintaro Itokawa <me@re-taro.dev>
* chore: @types/gtag.jsを追加

* feat: Google tag managerを埋め込むためのscriptやセレクタを実装した

* chore: 住所をセレクタブルにした

* chore: Google tag managerを_app.tsxで読み込んだ

* chore: Buttonイベントの取得

* chore: snapshotの更新
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* Update devDependencies

* chore: eslint configの依存が壊れていたのでそれの修正

* [skip ci] auto-fixable format by github-actions

* chore: コードを整形

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Rintaro Itokawa <me@re-taro.dev>
Co-authored-by: re-taro <re-taro@users.noreply.github.com>
* fix: CIの権限、及び処理を修正

よりセキュアにし、ドキュメントをもとに適切にcachingを施した

* fix: lhciのバージョンを修正

* fix: yarn add -g というオプションは存在しないぞ糸川
feat: design tokens transformerの追加
@github-actions
Copy link

@re-taro re-taro merged commit 3ee5764 into main Sep 18, 2022
- name: Test
run: yarn run test --no-cache
run-test-check:
if: ${{ ! failure() }}
Copy link
Member

Choose a reason for hiding this comment

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

ここってifだから${}いらなくない?
あとneedsがある時点でrun-testが落ちてると動かない気がする

Copy link
Member Author

@re-taro re-taro Sep 18, 2022

Choose a reason for hiding this comment

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

ここってifだから${}いらなくない?

! failure()という書き方はできないからいる

あとneedsがある時点でrun-testが落ちてると動かない気がする

無いとは思うけどrun-testがスキップされた場合にいる

Copy link
Member

Choose a reason for hiding this comment

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

オッケ了解です

re-taro added a commit that referenced this pull request Sep 18, 2022
CIの権限、及び処理を修正, PR templateを修正 (#176)
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.

4 participants