Skip to content

Conversation

@lizlooney
Copy link
Collaborator

Store a version number in all modules.

Even if we decide to upgrade when the project is loaded, it's easier to store the version number in every module.

Even if we decide to upgrade when the project is loaded, it's easier to store the version number in every module.
@lizlooney lizlooney requested a review from alan412 July 18, 2025 04:43
@lizlooney lizlooney marked this pull request as draft July 18, 2025 04:50
@lizlooney lizlooney marked this pull request as ready for review July 18, 2025 05:05
Copy link
Collaborator

@alan412 alan412 left a comment

Choose a reason for hiding this comment

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

Let's talk about this on Monday. I thought we had said just a project number. Also, I am beginning to wonder if we should just be saving our files as json rather than json as a comment in the python. I need to be reminded of the benefit of saving the python in the file.


// Types, constants, and functions related to modules, regardless of where the modules are stored.

export const CURRENT_VERSION = '0.1';
Copy link
Collaborator

Choose a reason for hiding this comment

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

I wonder if we should use the blockly version instead of having a version specific for the file format

methodsContent: string,
componentsContent: string): string {
componentsContent: string,
optVersionContent?: string): string {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Not sure we want this optional. When one is uploaded without it, we are giving it a default value.

@lizlooney lizlooney marked this pull request as draft July 19, 2025 04:45
@lizlooney lizlooney closed this Jul 22, 2025
@lizlooney lizlooney deleted the pr_version branch July 22, 2025 03:46
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