Skip to content

Add port option to qiita.config.json #43

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

Merged
merged 2 commits into from
Jul 26, 2023
Merged

Add port option to qiita.config.json #43

merged 2 commits into from
Jul 26, 2023

Conversation

getty104
Copy link
Member

What

qiita.config.jsonでポート番号を指定できるようにした。
portでポート番号を指定することができる(デフォルトは8888)
環境変数でポート番号を指定することができる仕様を削除した。

Why

  • 環境変数によるポート指定は開発用に用意していたもので、ユーザーが使うことを想定していなかったため削除した

Refs

@getty104 getty104 requested review from a team and ohakutsu and removed request for a team July 24, 2023 08:51
@@ -30,9 +31,8 @@ export async function startServer() {
app.use("*", express.static(path.join(__dirname, "../public/index.html")));

const server = createServer(app);
const port = process.env.QIITA_CLI_PORT
Copy link
Member

Choose a reason for hiding this comment

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

QIITA_CLI_PORT環境変数消します?

Copy link
Member Author

Choose a reason for hiding this comment

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

PRに記載の通り、削除で良いかなと思っています

Copy link
Member

@ohakutsu ohakutsu Jul 25, 2023

Choose a reason for hiding this comment

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

あ、見落としてました、
OKです!

Copy link
Member

@ohakutsu ohakutsu left a comment

Choose a reason for hiding this comment

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

LGTM!

@getty104 getty104 merged commit ac60f62 into main Jul 26, 2023
@getty104 getty104 deleted the add-port-config branch July 26, 2023 01:21
@ohakutsu ohakutsu mentioned this pull request Jul 28, 2023
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.

2 participants