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

Implement changes to track whether themes are outdated or not. #13

Merged
merged 5 commits into from
Nov 24, 2023

Conversation

MichaelPachec0
Copy link
Owner

This implements a change to conform to spicetify/spicetify-themes#1014. This now makes it easier to track if themes are outdated or not.

* Unwrap the check lambda, in preparation for the new changes from spicetify-themes.
* Reformat.
@MichaelPachec0 MichaelPachec0 changed the title Implement changes to track whether themes is outdated or not. Implement changes to track whether themes are outdated or not. Nov 18, 2023
@MichaelPachec0 MichaelPachec0 force-pushed the prepare-for-src-pr-1014 branch 11 times, most recently from ef53623 to b153b0f Compare November 19, 2023 07:54
* This is the bigger change of the series.
* This now checks the manfiest and includes an uncomitted version of manfiest.json that is part of upstream repo.
* A future commit will commit a new gh action that will check changes to this and commit them to  the repo.
* A future commit might also warn the user of which versions the outdated theme was known to work under.

Signed-off-by: Michael Pacheco <[email protected]>
@MichaelPachec0 MichaelPachec0 force-pushed the prepare-for-src-pr-1014 branch 2 times, most recently from 08d6978 to 9cb4e01 Compare November 19, 2023 08:45
@MichaelPachec0 MichaelPachec0 force-pushed the prepare-for-src-pr-1014 branch 3 times, most recently from 3cf399c to 7acfe29 Compare November 22, 2023 08:09
@MichaelPachec0 MichaelPachec0 marked this pull request as ready for review November 24, 2023 02:54
@MichaelPachec0 MichaelPachec0 merged commit 73c72af into dev Nov 24, 2023
2 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.

None yet

1 participant