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.

Sign up for GitHub

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

Update index.js #128

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update index.js #128

wants to merge 1 commit into from

Conversation

d-g-town
Copy link
Owner

@d-g-town d-g-town commented Nov 6, 2024

No description provided.

@d-g-town
Copy link
Owner Author

d-g-town commented Nov 6, 2024

/review

@d-g-town
Copy link
Owner Author

d-g-town commented Nov 6, 2024

/review test

@d-g-town
Copy link
Owner Author

d-g-town commented Nov 6, 2024

/review

2 similar comments
@d-g-town
Copy link
Owner Author

d-g-town commented Nov 6, 2024

/review

@d-g-town
Copy link
Owner Author

d-g-town commented Nov 6, 2024

/review

@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 6, 2024

👀 Review requested! I have added the needs-review label.

@d-g-town
Copy link
Owner Author

d-g-town commented Nov 6, 2024

/review

@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 6, 2024

👀 Review requested! I have added the needs-review label.

@d-g-town
Copy link
Owner Author

d-g-town commented Nov 6, 2024

/review

@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 6, 2024

👀 Review requested! I have added the needs-review label.

@d-g-town
Copy link
Owner Author

d-g-town commented Nov 6, 2024

/review

@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 6, 2024

👀 Review requested! I have added the needs-review label.

@alekcunn
Copy link

alekcunn commented Dec 4, 2024 •
edited
Loading

Just scrolling and saw your PR in search. Unsolicited advice but I think you will like it better using a ternary operator with a string

var sponsor = process.env.SPONSOR 
  ? "Tetris app is listening on port 3000! Brought to you by the one & only " + process.env.SPONSOR + ", of course." 
  : "Tetris app is listening on port 3000!";

Change it however you like.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
needs-review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@d-g-town @alekcunn