Skip to content

Simplify Testing in Codespaces#6

Merged
rcosta358 merged 16 commits intoliquid-java:mainfrom
rcosta358:automate-setup-codespaces
Sep 15, 2025
Merged

Simplify Testing in Codespaces#6
rcosta358 merged 16 commits intoliquid-java:mainfrom
rcosta358:automate-setup-codespaces

Conversation

@rcosta358
Copy link
Copy Markdown
Collaborator

Why?

The process of testing out LiquidJava with Codespaces involved multiple steps.
This is an attempt to make it easier and more friendly for the user.

Result

Now the user just needs two clicks to create the Codespace, which automatically opens the /examples/demo folder and installs the extension through the setup.sh script.

Limitations

Since we still need to rely on the VSIX file, this process is not as straightforward as it could be, and the only dev container command this worked was the postAttachCommand, since it is the only one that runs after VS Code fully loads and attaches to the container, allowing extensions to be installed. However, this command runs every time the Codespace is opened or a new terminal is created. For this reason, I also added a check to skip the extension installation process if it is already installed.

This will be improved in the future when the extension is published in the marketplace.

@CatarinaGamboa
Copy link
Copy Markdown
Collaborator

Is this ready for review @rcosta358 ?

@rcosta358
Copy link
Copy Markdown
Collaborator Author

Yup, I tested it on codespaces a couple times. I don't have the option to ask for review in this repository yet!

Copy link
Copy Markdown
Collaborator

@CatarinaGamboa CatarinaGamboa left a comment

Choose a reason for hiding this comment

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

If you tested it and it works, lgtm
I've added you as a collaborator, once you accept the request ou should be able to squash and merge it

@rcosta358 rcosta358 merged commit db87bea into liquid-java:main Sep 15, 2025
@rcosta358 rcosta358 deleted the automate-setup-codespaces branch September 15, 2025 19:10
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