Skip to content

LeBron#29

Open
Bhock91 wants to merge 6 commits intogigamonkey:mainfrom
Bhock91:main
Open

LeBron#29
Bhock91 wants to merge 6 commits intogigamonkey:mainfrom
Bhock91:main

Conversation

@Bhock91
Copy link
Copy Markdown

@Bhock91 Bhock91 commented Feb 23, 2026

made a lebron puzzle that requires some ball knowledge but very limited

Copy link
Copy Markdown
Owner

@gigamonkey gigamonkey left a comment

Choose a reason for hiding this comment

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

Make sure it runs then request another review.

Comment thread build/lebron-line.sh Outdated
set -euo pipefail

# shellcheck source=/dev/null
source "$(dirname "$0")/lebron-line.sh"
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This is weird. You are loading this file again from itself which is going to go into an infinite loop. Add

set -x

before this line and try to run it to see. (You can cd build and then run ./lebron-line.sh foo. You'll probably need to chmod +x lebron-line.sh too which is something you should do anyway.

(Oh, I think you were trying to copy the line from the other lines-based script. If so you should have copied it exactly:

source "$(dirname "$0")/lines.sh"

The point of that is to get all the functions defined in lines.sh that you want to use like ensure_lines, etc.

So that tells me you never actually ran this script. Either by itself or by adding it to the STEPS file and running make build. So you should make sure this actually runs and then request another review.

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