Skip to content

TemporaryLists#2528

Closed
R3d5t0n3GUY wants to merge 19 commits into
TurboWarp:masterfrom
R3d5t0n3GUY:master
Closed

TemporaryLists#2528
R3d5t0n3GUY wants to merge 19 commits into
TurboWarp:masterfrom
R3d5t0n3GUY:master

Conversation

@R3d5t0n3GUY

Copy link
Copy Markdown

Take Two for the Temporary Lists extension. I’ve removed the need to install Temporary Variables first, added some small optimizations, and provided more credit for references. Thanking @LilyMakesThings again for being my source of inspiration, and Mio for contributing to Temporary Variables

@R3d5t0n3GUY R3d5t0n3GUY requested a review from a team as a code owner June 6, 2026 00:57
@github-actions github-actions Bot added the pr: new extension Pull requests that add a new extension label Jun 6, 2026
@R3d5t0n3GUY

R3d5t0n3GUY commented Jun 6, 2026

Copy link
Copy Markdown
Author

I’m probably not going to bother with the Format and Validate bots, as long as the code is working

@GarboMuffin

Copy link
Copy Markdown
Member

They're not optional

@R3d5t0n3GUY

Copy link
Copy Markdown
Author

Last time, I didn’t get my code to a point where neither bot errored. I will try again soon

@Brackets-Coder

Copy link
Copy Markdown
Contributor

!format

@github-actions

github-actions Bot commented Jun 6, 2026

Copy link
Copy Markdown

The formatting bot couldn't push changes. Either maintainer edit permission is disabled or the pull request is from an organization/non-personal account. The author can run terminal command 'npm run format' manually to format all files.

@GarboMuffin

Copy link
Copy Markdown
Member

!format

It failed because the bot's email got unverified somehow

@R3d5t0n3GUY

R3d5t0n3GUY commented Jun 6, 2026

Copy link
Copy Markdown
Author

That's why I initially said that I probably wouldn't bother with the bots. Even after installing and enabling Prettier for my workspace, and triggering its format function, the bots STILL said my code wasn't formatted with Prettier

@R3d5t0n3GUY

R3d5t0n3GUY commented Jun 6, 2026

Copy link
Copy Markdown
Author

I apologize to everyone for the commit spamming, which is what I was trying to avoid this whole time. At least, the extension code is at a point where it can pass the required checks.

I'm apologizing for the commit spamming because, in the past, when I pushed commits to another project in rapid succession, some of the changes weren't registered to the environment, until I waited around five minutes BEFORE pushing more changes. And I was concerned that, if I committed too quickly, the changes that were required to pass the bot checks wouldn't be registered, which would probably have led to more frustration on my end.

@R3d5t0n3GUY

Copy link
Copy Markdown
Author

Dang it, the item [n] of (list) block isn't working

@R3d5t0n3GUY

Copy link
Copy Markdown
Author

!format

@R3d5t0n3GUY

Copy link
Copy Markdown
Author

!format

@Brackets-Coder

Copy link
Copy Markdown
Contributor

That's why I initially said that I probably wouldn't bother with the bots. Even after installing and enabling Prettier for my workspace, and triggering its format function, the bots STILL said my code wasn't formatted with Prettier

See https://docs.turbowarp.org/development/extensions/better-development-server. You can't just run the default npx prettier --write or similar on a file, the default settings don't match the version/settings for the repository. You have to run npm run format in the local copy of the repository on your computer

@Brackets-Coder

Brackets-Coder commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

Also, no need to apologize about commits, you're fine

@R3d5t0n3GUY

Copy link
Copy Markdown
Author

!format

@R3d5t0n3GUY

Copy link
Copy Markdown
Author

!format

@R3d5t0n3GUY R3d5t0n3GUY left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

!format

@R3d5t0n3GUY

Copy link
Copy Markdown
Author

!format

@R3d5t0n3GUY

Copy link
Copy Markdown
Author

So commenting !format via github.dev didn't summon DangoCat.

@github-actions

Copy link
Copy Markdown

The formatting bot didn't find any formatting issues. It currently only checks the extensions folder. The author or a maintainer can run terminal command 'npm run format' manually to format all files.

@R3d5t0n3GUY

Copy link
Copy Markdown
Author

!format

@R3d5t0n3GUY

Copy link
Copy Markdown
Author

I’m now closing this pull request because I’m no longer maintaining this version of the Temporary Lists extension. To stay up to date, head over to the version on my website. Once again, I’d like to thank @LilyMakesThings for being my source of inspiration for making this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: new extension Pull requests that add a new extension

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants