Skip to content

ci: cleanup workflows - #327

Merged
nekowinston merged 3 commits into
catppuccin:mainfrom
uncenter:ci-formatting
Nov 18, 2023
Merged

ci: cleanup workflows#327
nekowinston merged 3 commits into
catppuccin:mainfrom
uncenter:ci-formatting

Conversation

@uncenter

@uncenter uncenter commented Nov 17, 2023

Copy link
Copy Markdown
Member

Just a few touchups so everything looks nice.

@uncenter

uncenter commented Nov 17, 2023

Copy link
Copy Markdown
Member Author

Should I add # yaml-language-server: $schema=https://json.schemastore.org/github-workflow.json at the bottom of all of them or remove it from the few that have it?

@uncenter
uncenter marked this pull request as ready for review November 17, 2023 14:13
@uncenter uncenter changed the title ci: whitespace/formatting, add names to steps ci: cleanup workflows Nov 17, 2023
@nekowinston

nekowinston commented Nov 18, 2023

Copy link
Copy Markdown
Contributor

Thanks, I dropped the yamlls comments since .github/workflows/*.yml should now be correctly recognized in editors that have been updated within the last year.

@nekowinston
nekowinston merged commit 0814bb2 into catppuccin:main Nov 18, 2023
@uncenter

uncenter commented Nov 18, 2023 •
edited
Loading

Copy link
Copy Markdown
Member Author

Okay perfect, thanks!

@uncenter
uncenter deleted the ci-formatting branch November 18, 2023 03:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@nekowinston nekowinston nekowinston approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@uncenter @nekowinston