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

Add new flags to CI #8135

Open
Hiviexd opened this issue Oct 5, 2022 · 1 comment
Open

Add new flags to CI #8135

Hiviexd opened this issue Oct 5, 2022 · 1 comment
Assignees
Labels
area:meta non-article files

Comments

@Hiviexd
Copy link
Member

Hiviexd commented Oct 5, 2022

Currently, it's pretty easy to improperly write new flags as ::{FLAG=US}:: when in reality, it's missing spaces and should be ::{ FLAG=US }:: (happened with #8134)

A CI check would do wonders in not letting this error pass to production.

@Walavouchey Walavouchey added the area:meta non-article files label Oct 5, 2022
@TicClick TicClick mentioned this issue Oct 5, 2022
8 tasks
@TicClick
Copy link
Contributor

TicClick commented Oct 5, 2022

I'll reuse this issue to write down another thought from clayton (via)

[9:58 PM] clayton: all i can think of is forcing every custom span (the :: my content here :: thing) to be a flag since thats all we use em for
[9:58 PM] clayton: if u fail to make the span at all though it wouldnt help

@cl8n cl8n self-assigned this Nov 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:meta non-article files
Projects
None yet
Development

No branches or pull requests

4 participants