-
-
Notifications
You must be signed in to change notification settings - Fork 241
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
Flavor cupcake is missing several linters #4364
Comments
@bdovaz some choices have been made to make the image smaller ^^ Are they optimal, it's obviously debatable ^^
|
Ok, checking a little bit, of all the ones I mentioned. These are disabled by default, that's why cupcake doesn't include them:
@nvuillam I can create PR to add the 2 from powershell if you are ok with that and if you can tell me why it doesn't include the VALE linter we can fix that too. |
@bdovaz ok, if the powershell linters are not too heavy ;) |
Go :) |
Please read my message about VALE... |
@bdovaz flavors are overridden at linter level, you can add cupcake there :)
|
Is your feature request related to a problem? Please describe.
The flavor
cupcake
does not have the following linters:https://megalinter.io/beta/flavors/cupcake/
POWERSHELL_POWERSHELL -> powershell is cross platform (windows/linux/macos) and in infrastructure scripts can be common to have bash or powershell scripts side by side
POWERSHELL_POWERSHELL_FORMATTER -> powershell is cross platform (windows/linux/macos) and in infrastructure scripts can be common to have bash or powershell scripts side by side
JSON_ESLINT_PLUGIN_JSONC -> I don't know why this one is not included when the rest of the json linters are included.
MARKDOWN_REMARK_LINT -> I don't know why this one doesn't include it when the rest of markdown linters is included
SPELL_VALE -> I don't know why this is not included when the rest of the spell linters are included.
Describe the solution you'd like
That the
cupcake
flavor include the following previously mentioned linters.Describe alternatives you've considered
Use the
all
flavorAdditional context
Add any other context or screenshots about the feature request here.
cc @nvuillam @echoix
The text was updated successfully, but these errors were encountered: