-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Restructure this repository #71
Comments
To be fair this format is based on someone doing a JSON schema. They formatted it like this, but yeah, it could be improved looking at our use case. We could also do something like However, I'd like to see it as another repository. I think it would be better to do it from zero and rebuilding the history from it, but I don't want to remove every commit of this repository for the sake of starting over. We still have to support the branch method, though, for backwards purposes. Depreacate it, remove 6 months later, that kind of stuff. |
Sure, we can create another repo, how about |
Actually, this repository has a |
I'm not fond of "using branches for not-so-related content/history", so I rather not do that. Also, the default branch name is set per repository. What's in my mind is to just do |
Not changing the branch is fine with me.
I know.
That would work, although links to the legacy schemas would break. But… who uses those anyway? |
Also, GitHub warns against that:
Footnotes |
Again, I intend to do this so that But, yeah, I'm not sure who uses |
I think having a whole Action to copy the latest version of the schema to a branch isn't necessary. Here's what I think is a better plan:
For versioning, use tags instead of separate files.
It's easier for everyone to manage and lets us:
update-dist
Action anddist
branch entirelyTag contents can be referenced just like branch contents, in case anyone would want to use a specific version of the schema: https://raw.githubusercontent.com/ScratchAddons/ScratchAddons/refs/tags/v1.23.0/README.md
Also...
If it's the convention, please make tabs less than 8 spaces. I'd say 4 spaces is good.
How does that sound, @Hans5958 @WorldLanguages? I can help, too.
The text was updated successfully, but these errors were encountered: