Skip to content

typo - #1000

Closed
OutOfBedlam wants to merge 6 commits into
masterfrom
unknown repository
Closed

typo#1000
OutOfBedlam wants to merge 6 commits into
masterfrom
unknown repository

Conversation

@OutOfBedlam

Copy link
Copy Markdown
Contributor

What is this PR for?

bin/zeppelin.sh - typo fix

@jongyoul

Copy link
Copy Markdown
Member

@OutOfBedlam Thanks for the fix. LGTM. The failure of CI looks irrelevant.

@corneadoug

Copy link
Copy Markdown
Contributor

Merging if there is no more discussions

@corneadoug

Copy link
Copy Markdown
Contributor

@OutOfBedlam Could you reverse the last commit? I can't merge it now.
Also, when doing a PR, it is better to use a branch instead of master

@bzz

bzz commented Jun 15, 2016

Copy link
Copy Markdown
Member

Last 3 commits needs to be removed by using git interactive rebase, as it's been taken care of under #1009 then this branch can be rebased on latest master.

👍 for what @corneadoug suggests - using separate branch other then master, and rebase it frequetnly on to of the master is a good idea to avoid such commits in future.

It would also be great to have more descriptive PR title like Fix typo in bin/zeppelin.sh

Sign up for freeto 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.

4 participants

@OutOfBedlam@jongyoul@corneadoug@bzz