search

woodpecker-ci / woodpecker

Woodpecker is a simple, yet powerful CI/CD engine with great extensibility.
https://woodpecker-ci.org
Apache License 2.0
4.3k stars 371 forks source link

Add EditorConfig-Checker Plugin to docs #4371

Closed 6543 closed 1 week ago

6543 commented 1 week ago

REPO: https://codeberg.org/woodpecker-plugins/editorconfig-checker

DOCS: https://woodpecker-ci-woodpecker-pr-4371.surge.sh/plugins/EditorConfig%20Checker

woodpecker-bot commented 1 week ago

Deployment of preview was torn down

xoxys commented 1 week ago

Why? Im still not a fan of this wrapping approach as it adds a lot of overhead

xoxys commented 1 week ago

Ah this time its a real plugin, nevermind

pat-s commented 1 week ago

While I also don't like the idea of shell wrappers as plugins without any additional beef behind it, I think the overall idea is to have a large suite of "plug and play" plugins that are perceived as "secure" compared to normal steps that would execute the same job.

6543 commented 1 week ago

Plungins are not only for security ... they are also for beginners to not write scripts.

In this case the main benefit is feedback on sucess and the option to only lint changed files (via simple bool option)

As told the target audience of such plugins (prettier, pm, editorconfig, ...) are beginners not advanced users.

And if it exist we cam dogfood ...

I think I need to make an actual survay about this to support my argument with hard numbers 🤔.