Skip to content

added brillo support#140

Closed
unforseenconsequ3nces wants to merge 1 commit intojonascarpay:masterfrom
unforseenconsequ3nces:master
Closed

added brillo support#140
unforseenconsequ3nces wants to merge 1 commit intojonascarpay:masterfrom
unforseenconsequ3nces:master

Conversation

@unforseenconsequ3nces
Copy link

it works, Im not sure if I should do anything else here

@jonascarpay
Copy link
Owner

Thanks for the PR!
Is this repo is the best place for this package? apecs-gloss is really only here because it's an easy way to get graphics for the examples/tutorials. Similar to #136, maybe this is best put in a different repo, and then we can link to it from the README? Or, do you think we should switch from gloss to brillo?

@dpwiz
Copy link
Collaborator

dpwiz commented Nov 3, 2025

It was on my ask (=

I thought it would be a rather easy way to co-maintain the both things in one place.

However Brillo is using a different supported range (i.e. bytestring == 0.11) so I'm not sure now.

@unforseenconsequ3nces
Copy link
Author

I mean I can shove it in a different repository sure

@jonascarpay
Copy link
Owner

It was on my ask (=

I thought it would be a rather easy way to co-maintain the both things in one place.

However Brillo is using a different supported range (i.e. bytestring == 0.11) so I'm not sure now.

Ah, then I take it back, I trust your judgement on this :)

@unforseenconsequ3nces
Copy link
Author

Im working on moving this to here: https://github.com/fl215/apecs-brillo

@dpwiz dpwiz closed this Nov 4, 2025
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