src: ignore unset sentinel value for tailscale token

[aayushshah15]

Important

Skip tailnet join in joinTailnet() if BLACKSMITH_TAILSCALE_TOKEN is 'unset' and update string formatting in reportBuildPushActionFailure() calls.

• Behavior:
  • In joinTailnet(), skip tailnet join if BLACKSMITH_TAILSCALE_TOKEN is 'unset'.
• Misc:
  • Change string formatting in reportBuildPushActionFailure() calls in startBlacksmithBuilder() and actionsToolkit.run().

^{This description was created by for 77a6a48. It will automatically update as commits are pushed.}

[aayushshah15]

https://github.com/useblacksmith/web/pull/801

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to b0458c2 in 10 seconds

More details

• Looked at 40 lines of code in 1 files
• Skipped 2 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. src/main.ts:32

• Draft comment:
  Checking for 'unset' is a good addition to ensure the token is not mistakenly used when not properly set.
• Reason this comment was not posted:
  Confidence changes required: 0%
  The change in the PR is to ignore the 'unset' value for the BLACKSMITH_TAILSCALE_TOKEN environment variable. This is a valid change to ensure that the token is not mistakenly used when it is not set properly.

Workflow ID: wflow_OvtlDLAiZO2xxHY9

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

[ellipsis-dev]

👍 Looks good to me! Incremental review on 77a6a48 in 11 seconds

More details

• Looked at 40 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 2 drafted comments based on config settings.

1. src/main.ts:118

• Draft comment:
  String formatting updated to use single quotes, consistent with the PR description.
• Reason this comment was not posted:
  Confidence changes required: 10%
  The PR description mentions updating string formatting in reportBuildPushActionFailure() calls. The changes are consistent with this intent, replacing double quotes with single quotes for string literals.

2. src/main.ts:409

• Draft comment:
  String formatting updated to use single quotes, consistent with the PR description.
• Reason this comment was not posted:
  Confidence changes required: 10%
  The PR description mentions updating string formatting in reportBuildPushActionFailure() calls. The changes are consistent with this intent, replacing double quotes with single quotes for string literals.

Workflow ID: wflow_1vTPG4X5FXnXhvY0

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

[ellipsis-dev]

⚠️ This PR is too big for Ellipsis, but support for larger PRs is coming soon. If you want us to prioritize this feature, let us know at [email protected]