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.

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

example用userのディレクトリ名の修正 #24

Merged
merged 1 commit into from
Nov 22, 2021

Conversation

sksat
Copy link
Collaborator

@sksat sksat commented Nov 22, 2021

概要

Examples/maximum_user_sample_for_s2e_buildは長すぎるしminimumのtypoっぽい

Issue

@sksat sksat requested a review from meltingrabbit November 22, 2021 09:46
@sksat sksat self-assigned this Nov 22, 2021
@meltingrabbit meltingrabbit added the priority::medium priority medium label Nov 22, 2021
Copy link
Collaborator

@meltingrabbit meltingrabbit left a comment

Choose a reason for hiding this comment

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

muximumはタイポでした・・・

@sksat
Copy link
Collaborator Author

sksat commented Nov 22, 2021

ですよね.びっくりした.

@meltingrabbit
Copy link
Collaborator

パス名は長いなぁと思ってたけど,まあそのまま愚直に書くか~,という感じだった.

これでもわかるしスッキリするのでこれで良さそう.

@sksat
Copy link
Collaborator Author

sksat commented Nov 22, 2021

sampleであることは自明ですし,buildは冗長なんですよね.あとそういう情報はREADMEにちゃんと書いてあれば問題無い.

@meltingrabbit
Copy link
Collaborator

sampleであることは自明ですし

Examplesの下にあるので,自明だね.

LGTMなので,マージどうぞ~

@sksat sksat merged commit 0d79658 into develop Nov 22, 2021
@sksat sksat deleted the feature/fix-example-dir-name branch November 22, 2021 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority::medium priority medium
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants