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

Issue#4 #9

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Issue#4 #9

wants to merge 3 commits into from

Conversation

legokichi
Copy link

#4

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

現行の方式では/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__という安易な形にして良いのか」等、問題があります。

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

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.

1 participant