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.

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

codespacesの設定 #54

Merged
merged 3 commits into from
Jun 17, 2023
Merged

codespacesの設定 #54

merged 3 commits into from
Jun 17, 2023

Conversation

Yanai1005
Copy link
Collaborator

@Yanai1005 Yanai1005 commented Jun 10, 2023 •
edited
Loading

@yukinissie レビューとマージお願いします！
デフォルトの設定でCode Spacesを起動すると最新のNodeのバージョンになるので，設定を追加しました．

@Yanai1005 Yanai1005 changed the title codespacesno codespacesの設定 Jun 10, 2023
@Yanai1005 Yanai1005 requested a review from yukinissie June 10, 2023 08:05
@Yanai1005 Yanai1005 self-assigned this Jun 10, 2023
@Yanai1005 Yanai1005 added the enhancement New feature or request label Jun 10, 2023
@yukinissie
Copy link
Collaborator

@Yanai1005 プルリクの向き先間違えてないよねー？ inaokaくんのリポジトリに貢献するって意味では間違ってないけどー

@yukinissie
Copy link
Collaborator

yukinissie commented Jun 10, 2023 •
edited
Loading

デフォルトの設定でCode Spacesを起動すると最新のNodeのバージョンになるので，設定を追加しました．

これはプロジェクトのNodeバージョンでCodeSpacesを起動するようにしたいということでOK？（プロジェクトのNodeバージョンってなんじゃろ？）

@Yanai1005
Copy link
Collaborator Author

@yukinissie 間違ってないですねー、稲岡先輩がやったので問題ないと思いますw

@takatoshiinaoka
Copy link
Owner

onkのNodeのバージョンは16.xです〜

yukinissie
yukinissie approved these changes Jun 10, 2023
View reviewed changes
Copy link
Collaborator

@yukinissie yukinissie left a comment

Choose a reason for hiding this comment

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

LGTMです！勉強になります！

@yukinissie
Copy link
Collaborator

あ！でもCIが落ちてるの気になる！

@takatoshiinaoka
Copy link
Owner

稲岡アカウント+稲岡デプロイトークン=稲岡Azure
柳井アカウント+柳井デプロイトークン=柳井Azure

これから考えると、柳井くんのAzure環境にデプロイするトークンのまま、稲岡のgithubアカウントでデプロイしようとしたから権限エラーが出た的な？？

@takatoshiinaoka
Copy link
Owner

ちゃんとエラー見てないので憶測ですがw

@yukinissie
Copy link
Collaborator

yukinissie commented Jun 10, 2023 •
edited
Loading

@takatoshiinaoka
単純に有効なトークンがAZURE_STATIC_WEB_APPS_API_TOKEN_GREEN_WAVE_09FCC8700に登録されていない説あるよ。ほかのPRも同じワークフローが落ちているみたいだし

@takatoshiinaoka
Copy link
Owner

これっぽい？
https://qiita.com/masato_makino/items/9ed495538ca972551dc6

@takatoshiinaoka takatoshiinaoka self-requested a review June 11, 2023 14:10
@takatoshiinaoka
Copy link
Owner

フォークしたリポジトリからのActionsはセキュリティ面を考慮して勝手に実行できないようになってるみたい？
だけど、承認ボタンがどこにも見当たらない...
https://docs.github.com/ja/actions/managing-workflow-runs/approving-workflow-runs-from-public-forks

@takatoshiinaoka
Copy link
Owner

Azure/static-web-apps#679

@takatoshiinaoka
Copy link
Owner

takatoshiinaoka commented Jun 16, 2023 •
edited
Loading

@yukinissie
同じリポジトリ内のブランチだとデプロイ成功するので、柳井君のリポジトリからだと権限が無い気がします〜
#57

@yukinissie
Copy link
Collaborator

@takatoshiinaoka 権限ない系は設定でなんとかできないかな？

@yukinissie
Copy link
Collaborator

@takatoshiinaoka @Yanai1005 @
まあ今回のPRと関係のないエラーだからマージしちゃっていいと思います

@takatoshiinaoka takatoshiinaoka merged commit e5a7d77 into takatoshiinaoka:main Jun 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yukinissie yukinissie yukinissie approved these changes

@takatoshiinaoka takatoshiinaoka takatoshiinaoka approved these changes

Assignees

@Yanai1005 Yanai1005

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Yanai1005 @yukinissie @takatoshiinaoka