Skip to content

Adapt e2e tests to the new foundry-rpc #130

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

Merged
merged 3 commits into from
Feb 13, 2020
Merged

Conversation

MSNTCS
Copy link
Contributor

@MSNTCS MSNTCS commented Jan 30, 2020

No description provided.

@MSNTCS MSNTCS requested a review from majecty January 31, 2020 01:58
@MSNTCS MSNTCS force-pushed the master branch 3 times, most recently from 49012bd to d3baef1 Compare February 1, 2020 11:40
Copy link
Contributor

@majecty majecty left a comment

Choose a reason for hiding this comment

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

Please be careful about what you added in a commit.
You must not add unrelated code change in a commit.

You can use git add -p command or CLion's tool to select only the diff you need to add in a commit.

These links will help you:
https://gist.github.com/mattlewissf/9958704
https://blog.jetbrains.com/idea/2018/02/intellij-idea-2018-1-eap-partial-git-commits-and-more/

Also, I wrote a help message in our wiki about this:
https://github.com/kodebox-io/kodebox-internal/wiki/Tips-for-Developers#selectively-stage-your-work

@MSNTCS MSNTCS closed this Feb 3, 2020
@MSNTCS MSNTCS reopened this Feb 3, 2020
@MSNTCS MSNTCS force-pushed the master branch 3 times, most recently from f456beb to 76a396b Compare February 3, 2020 05:47
@MSNTCS MSNTCS changed the title [wip] Adapt e2e tests to the new foundry-rpc [WIP] Adapt e2e tests to the new foundry-rpc Feb 3, 2020
Copy link
Contributor

@majecty majecty left a comment

Choose a reason for hiding this comment

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

I'm still reviewing. I'll continue to review this PR tomorrow.

@MSNTCS MSNTCS changed the title [WIP] Adapt e2e tests to the new foundry-rpc Adapt e2e tests to the new foundry-rpc Feb 5, 2020
@MSNTCS MSNTCS force-pushed the master branch 2 times, most recently from ebb3bc6 to 9c46d4c Compare February 6, 2020 11:42
majecty
majecty previously approved these changes Feb 10, 2020
Copy link
Contributor

@majecty majecty left a comment

Choose a reason for hiding this comment

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

LGTM.
Could you squash all commits and rebase on the foundry/master?

@MSNTCS MSNTCS force-pushed the master branch 3 times, most recently from 7bed25e to 1f1f893 Compare February 11, 2020 08:41
majecty
majecty previously approved these changes Feb 11, 2020
@majecty majecty merged commit 027bc13 into CodeChain-io:master Feb 13, 2020
@MSNTCS MSNTCS mentioned this pull request Feb 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants