Issue#4#9

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open

legokichi wants to merge3 commits intouupaa:master

base:master

Choose a base branch

Open

Conversation

Copy link

この問題に関する提案です。

現行の方式では/github.com(.)uupaa/のuupaaを置換していました。
しかしこの方法では、先のissue#4に挙げたような問題が発生していました。

text=text.replace(/github.com(.)uupaa/g,function(_,sep){return"github.com"+sep+githubUserName;});

そこで、置換対象を__GITHUB_USER_NAME__というマクロ変数を利用して問題の解決を試みます。

text=text.replace(/github.com(.)uupaa/g,function(_,sep){return"github.com"+sep+githubUserName;});

この方式も「マクロ変数名を__HOGE__という安易な形にして良いのか」等、問題があります。

ひとつの提案としてご査収ください。

Legokichi Duckscallion added3 commits

issue#4

bb4c0c5

remove comment

4de3ea2

remove whitespace

1a9630e

Labels

None yet

1 participant