Open
Conversation
9b776ad to
df9d9a1
Compare
maconaco
reviewed
Oct 22, 2020
| int main() { | ||
| std::cout << "Hello" << std::endl; | ||
| } | ||
| ``` |
Contributor
There was a problem hiding this comment.
このCommit時にindentでなく言語識別子付きの三連バッククォートでコードブロック記述は残してはいけなかった
df9d9a1 to
a226bc3
Compare
Contributor
にゃおさんの作業してたブランチに移動
rebase作業
|
Member
|
LGTM! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
まこさんのリポジトリにC++のHello World的なコードを追加するブランチを作成して、コミットを重ねていたのだがそもそも最初のコミットからして動かないコードを書いてしまっていた!あわててコードを修正するコミットをしたが、これでは最初のコミットが『どのコミットの状態でもコードはコンパイル可能な状態であるべき』という原則(※と私は思ってる)に違反してしまっている!私の代わりに
git rebase -iを使って直して欲しい!今の状態
の順にコミットが並んでいる。
変更して欲しい内容
『必要なヘッダの記述が抜けていたため追記修正 9b776ad』で行った修正を『READMEに C++ のコード片を追加 08fbfb4』に合体させ、他のコミットはそのままにする。
最終的なコミットは
の2つになる。(それぞれのコミットが持つ diff は変化する)