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.

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

fixed various syntax errors keeping the code from functioning: #1

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

scottvr
Copy link

@scottvr scottvr commented Mar 20, 2023

fixed various syntax errors keeping the code from functioning: missing "scripts." on line 13, (un)indent on lines 611 and 618 of inspector. various ] and ) issues on ln 245, 328, 445, 486, and 599, change += to + on ln 259 of solver.

I wanted to use your fork of this extension after reading the dialog between you and tkalayci7 and being disappointed in how the solver issue was resolved (or wasn't), but I found that in its present state the code won't run due to lots of minor syntax typos and although I don't know if you even still care aboutt this script I figured that if you do get around to trying to run it again eventually, you'd have to resolve these same issues and since I've already done the work, I could save you some time with this PR.

Nothing functional or substantional, just syntax to get it to stop bailing with errors.

…g "scripts." on line 13, (un)indent on lines 611 and 618 of inspector. various ] and ) issues on ln 245, 328, 445, 486, and 599, change += to + on ln 259 of solver.
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.

1 participant