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

Fix API Secret, README, grammatic errors #10

Merged
merged 1 commit into from
Oct 20, 2013
Merged

Fix API Secret, README, grammatic errors #10

merged 1 commit into from
Oct 20, 2013

Conversation

maddie
Copy link

@maddie maddie commented Oct 20, 2013

之前因为我的vim自动切掉了每一行最后的空格所以弄乱了README的markdown格式,修好了~
另外我说那个API Secret提示的问题,我试着这样改了一下就ok了,你看看~~
curl的output太乱了,加了--silent来把curl的传输过程省掉了

最后出来的json估计可以parse一下变得更加好看,不过我只知道python怎么整-.-~

Commit Message:

Fix some grammar errors, and fix README markdown newlines
Also fix API Secret input prompt so it now works properly.
Added --silent to curl to omit the transmission log, should look a
little cleaner.

Fix some grammar errors, and fix README markdown newlines
Also fix API Secret input prompt so it now works properly.
Added --silent to curl to omit the transmission log, should look a
little cleaner.
@oott123
Copy link
Owner

oott123 commented Oct 20, 2013

唔……这个pull request太乱了,导致我分不清哪个是为了修正断行哪个是为了修正app secret的错误……
另外curl不加slient就是为了上传的时候能够看到进度啊~所以还是保留比较好0.0嗯
我先合并进来,再进行修改吧。

oott123 added a commit that referenced this pull request Oct 20, 2013
Fix API Secret, README, grammatic errors
@oott123 oott123 merged commit c4f61e4 into oott123:master Oct 20, 2013
oott123 added a commit that referenced this pull request Oct 20, 2013
@maddie
Copy link
Author

maddie commented Oct 21, 2013

hmmm....

app secret那里其实我就是在[ ]中间把'.BPCSU_SEC.'加上了,然后就不跳过了=.=

@oott123
Copy link
Owner

oott123 commented Oct 21, 2013

并不是这样的=-=

issue #2 里有提到这个问题。我修复之后合并了一个分支,因为误操作又将错误的修复回滚了。

大致的意思是,PHP中的Where doc(也就是<<<EOF)的结束符EOF需要顶格写,我的编辑器给它加了一个tab符,自然就出问题了。

@maddie
Copy link
Author

maddie commented Oct 30, 2013

好吧,表示对PHP一窍不通 T.T

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants