add camelize , decamelize flatten function to js.md #36

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

Jump to bottom

Open

gnipbao wants to merge 3 commits into jsfront:master

from gnipbao:master

Open

add camelize , decamelize flatten function to js.md #36

gnipbao wants to merge 3 commits into jsfront:master from gnipbao:master

Conversation

@gnipbao

Copy link

@gnipbao gnipbao commented Dec 16, 2017

add camelize , decamelize flatten function to js.md (ps:test pr):dog:

gnipbao added 3 commits

December 16, 2017 22:37

@gnipbao


 add camelize , decamelize flatten function to js.md

d31f64b

@gnipbao


 add camelize , decamelize flatten function to js.md

ffaa88f

@gnipbao


 merge js.md

9a6ba19

S1ngS1ng

S1ngS1ng suggested changes

Dec 18, 2017

View reviewed changes

js.md

function decamelize(str, separator){

separator = typeof separator === 'undefined' ? '_' : separator;

return str

.replace(/([a-z\d])([A-Z])/g, '1ドル' + separator + '2ドル')

Copy link

@S1ngS1ng S1ngS1ng Dec 18, 2017 •

edited

Loading

Choose a reason for hiding this comment

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

谢谢你的 pr。请把这里的缩进调整下吧。像上面一样,四个就好。文件其他位置也有缩进不对的。我有空了再调吧

js.md

* @param separator Separator for the new decamelized string.

function decamelize(str, separator){

separator = typeof separator === 'undefined' ? '_' : separator;

Copy link

@S1ngS1ng S1ngS1ng Dec 18, 2017

Choose a reason for hiding this comment

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

这里也是

js.md

```js

function flatten(arr){

var i,ret=[], len = arr.length;

for(i=0;i<len;i++){

Copy link

@S1ngS1ng S1ngS1ng Dec 18, 2017

Choose a reason for hiding this comment

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

还有这里。谢谢

Labels

None yet

2 participants

@gnipbao @S1ngS1ng

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

add camelize , decamelize flatten function to js.md #36

Are you sure you want to change the base?

add camelize , decamelize flatten function to js.md #36

Uh oh!

Conversation

@gnipbao gnipbao commented Dec 16, 2017

Uh oh!

@S1ngS1ng S1ngS1ng Dec 18, 2017 •

edited

Loading

Uh oh!

Choose a reason for hiding this comment

Uh oh!

@S1ngS1ng S1ngS1ng Dec 18, 2017

Choose a reason for hiding this comment

Uh oh!

@S1ngS1ng S1ngS1ng Dec 18, 2017

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

add camelize , decamelize flatten function to js.md #36

Are you sure you want to change the base?

add camelize , decamelize flatten function to js.md #36

Uh oh!

Conversation

@gnipbao gnipbao commented Dec 16, 2017

Uh oh!

@S1ngS1ng S1ngS1ng Dec 18, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

@S1ngS1ng S1ngS1ng Dec 18, 2017

Choose a reason for hiding this comment

Uh oh!

@S1ngS1ng S1ngS1ng Dec 18, 2017

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

@S1ngS1ng S1ngS1ng Dec 18, 2017 •

edited

Loading