Skip to content

Conversation

@nbogie
Copy link
Collaborator

@nbogie nbogie commented Dec 10, 2025

I've made some simplifications to get this merged before the codebase moves on again:

  • Ignores p5.js version response (the package will have a dependency on the latest version of p5.js)
  • Ignores CDN/local prompt response - p5 lib will always be downloaded for development by NPM, will include types.
  • Both global and instance mode use the same template (which i've placed in basic-global-ts. For now, user will have to update their html file to point at the instance-mode sketch if that's what they want.

Whilst these are imperfect, it's a start, it works, and is far better than the previous ts template.

@nbogie
Copy link
Collaborator Author

nbogie commented Dec 10, 2025

updated now to use the same template directory contents for both global and instance mode.

@nbogie
Copy link
Collaborator Author

nbogie commented Dec 10, 2025

resolves issue #28

@SableRaf
Copy link
Owner

SableRaf commented Dec 20, 2025

I've tried the branch and it works when specifying --language typescript (though it does not seem to ignore the CDN/local or global/instance prompts).

image

However, the custom path fails:

image

This may not be entirely relevant though since I made some changes since then, and I see there are now conflicts with the main branch again. Sorry it took so long for me to find a moment to look at it.

What would be the easiest path to getting your work merged now?

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.

3 participants