Skip to content
This repository was archived by the owner on Feb 20, 2025. It is now read-only.

Conversation

@Davilarek
Copy link
Member

Adds auto conversion of working theme files to our format.

Adds auto conversion of working theme files to our format
@Davilarek Davilarek requested a review from zrodevkaan April 14, 2024 21:46
@zrodevkaan zrodevkaan marked this pull request as ready for review April 14, 2024 22:35
@zrodevkaan
Copy link
Contributor

Is this completed?
@Davilarek

@Davilarek
Copy link
Member Author

No, stage 1 needs to return less props, will do later. Currently working on something else.

@zrodevkaan
Copy link
Contributor

Alright, lmk when the narrowing down the props is completed.

@zrodevkaan zrodevkaan marked this pull request as draft April 14, 2024 22:39
@zrodevkaan
Copy link
Contributor

Status on this ?

@Davilarek
Copy link
Member Author

I need more complex theme files to test

@DaBluLite
Copy link

@zrodevkaan
Copy link
Contributor

I'm guessing this was converted?

@DaBluLite
Copy link

No, I mean, don't you want some kind of theme to try reverse lookup on?

@zrodevkaan
Copy link
Contributor

zrodevkaan commented Apr 29, 2024

Ooh I see, dav will be happy to use that. Thanks DaBluLite

I never saw the message he sent so I just guessed.

@Davilarek
Copy link
Member Author

DaBluLite/Cyan@master/import2.css Try this

Thanks, after quick look at the output file I noticed some conversion errors. I will look into this

@Davilarek
Copy link
Member Author

@zrodevkaan please review this. It's almost ready to merge, I just need more feedback
@DaBluLite I would be thankful if you could test this PR on some theme files and then convert them back to regular theme files to see if something is wrong

@zrodevkaan
Copy link
Contributor

code looks fine

@Davilarek
Copy link
Member Author

I would like to merge this but no one is interested in testing

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants