Skip to content

feat(import-styles): init - #399

Merged
isabelroses merged 33 commits into
mainfrom
ci/gen-import
Dec 15, 2023
Merged

feat(import-styles): init#399
isabelroses merged 33 commits into
mainfrom
ci/gen-import

Conversation

@isabelroses

Copy link
Copy Markdown
Member
  • Allows for a mass import of userstyles.
  • We can enforce settings upon users making our lives easier.

Comment thread .github/workflows/generate-import.yml Outdated
Comment thread README.md Outdated
Co-authored-by: uncenter <47499684+uncenter@users.noreply.github.com>
Comment thread deno.json Outdated
Comment thread scripts/import-styles/main.ts Outdated
isabelroses and others added 2 commits December 13, 2023 22:22
Co-authored-by: uncenter <47499684+uncenter@users.noreply.github.com>
Comment thread scripts/import-styles/main.ts Outdated
isabelroses and others added 3 commits December 13, 2023 22:24
Co-authored-by: uncenter <47499684+uncenter@users.noreply.github.com>
Comment thread scripts/import-styles/main.ts Outdated
@isabelroses

isabelroses commented Dec 14, 2023

Copy link
Copy Markdown
Member Author

This PR will need refactoring as of #400 being merged

Comment thread README.md Outdated
Comment thread scripts/import-styles/main.ts Outdated
Comment thread .github/workflows/generate-import.yml Outdated
Comment thread styles/github/catppuccin.user.css
@github-actions github-actions Bot removed the github GitHub label Dec 15, 2023
Comment thread .github/workflows/lint-build.yml
Comment thread scripts/import-styles/main.ts Outdated
Comment thread scripts/import-styles/main.ts Outdated
@nekowinston

This comment was marked as resolved.

@nekowinston nekowinston left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.
The last thing we need to do is create that release that we're continuously uploading to.
import-styles for the name seems a bit vague? stylus-backup? latest? nightly? I don't know really.

@isabelroses

isabelroses commented Dec 15, 2023

Copy link
Copy Markdown
Member Author

import-styles for the name seems a bit vague? stylus-backup? latest? nightly? I don't know really.

I quite like what @uncenter been referring to it as, "stylus export", beyond that i'm unsure either.

@sgoudham

sgoudham commented Dec 15, 2023

Copy link
Copy Markdown
Contributor

Why can't we name it what it is, all-userstyles-export? all-userstyles-backup? all-userstyles?
Just throwing out suggestions in case it sparks a lightbulb idea for anyone else.

@uncenter

Copy link
Copy Markdown
Member

I vote for all-userstyles-export!

@isabelroses

Copy link
Copy Markdown
Member Author

I vote for all-userstyles-export!

This makes sense also has my vote.

@nekowinston

Copy link
Copy Markdown
Contributor

i don't care

Comment thread README.md Outdated
@nekowinston

nekowinston commented Dec 15, 2023

Copy link
Copy Markdown
Contributor

I've marked it as a pre-release, tag is up: https://github.com/catppuccin/userstyles/releases/tag/all-userstyles-export

The tag doesn't matter, we won't have to update it. Should be good to merge.

@isabelroses
isabelroses merged commit a789798 into main Dec 15, 2023
@isabelroses
isabelroses deleted the ci/gen-import branch December 15, 2023 15:37
@isabelroses

Copy link
Copy Markdown
Member Author

The CI is failing https://github.com/catppuccin/userstyles/actions/runs/7224061024/job/19684480303, due to a lack of GH_TOKEN.

@nekowinston

Copy link
Copy Markdown
Contributor

And here I was worried that it was going to work in a single commit 🙃

Would you mind fixing it?

@isabelroses

Copy link
Copy Markdown
Member Author

Would you mind fixing it?

I would have done it already but i had to run. Should be able to do it now.

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.

5 participants