[PR #51] Match internal and external port to 4000 for sanity #83

New Issue

saavagebueno · 2025-11-20T04:15:31-05:00

saavagebueno commented

2025-11-20 04:15:31 -05:00

Original Pull Request: https://github.com/BretFisher/jekyll-serve/pull/51

State: closed
Merged: Yes

Closes #50

This was a simple bug, and I realized were two ways to solve it:

Declare an extra enviorment variable for host port and set the dockerfile accordingly
Map both the container and host ports to 4000.

I chose the second hassle-free solution.

p.s: The solution was drafted without the consulation of the maintainer 😨 , feel free to make any suggestions.

**Original Pull Request:** https://github.com/BretFisher/jekyll-serve/pull/51 **State:** closed **Merged:** Yes --- Closes #50 This was a simple bug, and I realized were two ways to solve it: - Declare an extra enviorment variable for host port and set the dockerfile accordingly - Map both the container and host ports to 4000. I chose the second hassle-free solution. p.s: The solution was drafted without the consulation of the maintainer 😨 , feel free to make any suggestions.

saavagebueno added the pull-request label 2025-11-20 04:15:31 -05:00

saavagebueno closed this issue

2025-11-20 04:15:31 -05:00

saavagebueno referenced this issue

2025-11-20 04:15:37 -05:00

[PR #83] move back to ruby 3.1 #102

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/jekyll-serve#83