Skip to content

Conversation

@itowlson
Copy link
Collaborator

This provides TOML environment definitions for Spin 3.2 and 3.4. (It will also need a definition for 3.5, once #3295 is agreed.)

Proposed plan of attack:

  • Once Workflow for publishing spin:up WIT package #3300 merges, we can run that against the 3.4 and 3.2 branches to get those WITs into the default spinframework.dev registry.
  • Then we can test these TOMLs using local files against all of our templates and any known good applications. This is important because during dev I ran into problems with apps that targeted older worlds and we want to make sure that these (within reason) do not give spurious errors. My dev test suggested that we might need to retrofit some compat stuff to the top level worlds which I really hope we don't have to do but well this is where we find out I guess.
  • Then we can merge them. mumble mumble automation mumble CI mumble muttered cursing
  • Then we can push them to the default OCI registry.
  • Then we can test them again using just env names instead of file references. Which will work first time and will produce no mysterious inconsistencies, none do you hear me, but you know, good form.
  • And then we can tell people about them!

@itowlson itowlson marked this pull request as draft October 14, 2025 01:02
@itowlson
Copy link
Collaborator Author

#3293 had an open question about where we want to keep these but it's not had much traction so I'm parking them here for now but if people have other preferences then whatever

@itowlson itowlson force-pushed the environment-definitions-starter-pack branch from 7efdfd5 to c4d4c36 Compare October 22, 2025 00:38
@itowlson itowlson force-pushed the environment-definitions-starter-pack branch from c4d4c36 to 39da5a2 Compare October 22, 2025 19:55
@itowlson itowlson marked this pull request as ready for review October 23, 2025 20:24
@itowlson
Copy link
Collaborator Author

@fibonacci1729 with the 3.2-2.5 packages uploaded I think this is ready to go but I am not sure if the script/workflow is right or how to test it. The script works (mutatis mutandis) locally though...

@itowlson itowlson force-pushed the environment-definitions-starter-pack branch from 23b8f79 to a08a2ed Compare October 23, 2025 20:44
Copy link
Collaborator

@fibonacci1729 fibonacci1729 left a comment

Choose a reason for hiding this comment

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

LGTM!

@itowlson itowlson enabled auto-merge October 23, 2025 21:18
@itowlson itowlson merged commit d1b931c into spinframework:main Oct 23, 2025
17 checks passed
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