Skip to content
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

Remove legacy templates #36

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Remove legacy templates #36

wants to merge 2 commits into from

Conversation

leptos-null
Copy link
Member

What does this implement/fix? Explain your changes.

In an effort for the default set of templates to be as helpful as possible, this PR removes templates that target older operating systems and configurations.

The templates being removed in this PR will be moved to a Theos module. This PR will only be merged after that module is available. This PR represents the place to discuss which templates move over to that module.

(Based on conversation in #31 (comment))

Important note: #31 effectively removes iphone/application from this repository (as it is legacy). That template is not removed in this PR since #31 already does. The iphone/application template will be moved over to the "legacy templates" module.

Does this close any currently open issues?

No

Any relevant logs, error output, etc?

No

Any other comments?

There is some additional discussion in the Theos Discord on this topic here: https://discord.com/channels/811490080278839327/811494949849661490/1332595828098994186

Copy link
Member

@L1ghtmann L1ghtmann left a comment

Choose a reason for hiding this comment

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

Thanks for spearheading this, Leptos! Only comment for the pr is we should probably remove the templates from the README list too. Tangentially, we should update the docs to reflect the changes. I can help with some or all of that if desired!

@leptos-null
Copy link
Member Author

Great call, thank you!
I'll add the README change to this PR.

That would be helpful, yes please - may you (or anyone else) add the docs change?

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