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

fix Dockerfile #1

Merged
merged 2 commits into from
Oct 23, 2023
Merged

fix Dockerfile #1

merged 2 commits into from
Oct 23, 2023

Conversation

Mogyuchi
Copy link
Collaborator

No description provided.

@Mogyuchi Mogyuchi marked this pull request as draft October 18, 2023 10:16
cm-ayf
cm-ayf reviewed Oct 20, 2023
View reviewed changes
Dockerfile Show resolved Hide resolved
@Mogyuchi
Copy link
Collaborator Author

#2 の変更に依存しているためブロッキングしています。

@cm-ayf
Copy link
Collaborator

cm-ayf commented Oct 21, 2023

#2 merged

@Mogyuchi Mogyuchi marked this pull request as ready for review October 21, 2023 11:32
@cm-ayf cm-ayf requested a review from InkoHX October 21, 2023 11:35
InkoHX
InkoHX reviewed Oct 21, 2023
View reviewed changes
Dockerfile Outdated Show resolved Hide resolved
Dockerfile Outdated Show resolved Hide resolved
@Mogyuchi @InkoHX
fix Dockerfile
eeee00a 
- npm ci を使用するように変更。
- 記法統一

Co-authored-by: InkoHX <[email protected]>
@Mogyuchi Mogyuchi requested a review from InkoHX October 23, 2023 10:35
@cm-ayf cm-ayf merged commit 1bf7d6e into discordjs-japan:main Oct 23, 2023
1 check passed
@Mogyuchi Mogyuchi deleted the fix-dockerfile branch October 23, 2023 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cm-ayf cm-ayf cm-ayf approved these changes

@InkoHX InkoHX InkoHX approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Mogyuchi @cm-ayf @InkoHX