jbang icon indicating copy to clipboard operation
jbang copied to clipboard

Clarification on //DEPS

Open gunnarmorling opened this issue 5 months ago • 1 comments

This just cost me five min and some grey hair ;)

gunnarmorling avatar Jun 16 '25 13:06 gunnarmorling

While I do understand your frustration, the thing is that there should be no space for any of the directives, not just //DEPS. So how do we make that clear? (Hopefully without having to repeat that sentence for each and every directive)

quintesse avatar Jun 16 '25 16:06 quintesse

fwiw its quite intentional for two reasons:

a) faster to search/match on ^//DEPS b) the space lets you comment out the directive. if space was not meaningfull you wouldn't be able to comment it out.

maxandersen avatar Jun 18 '25 06:06 maxandersen

worth putting it here as the first/direct place its listed - we probably should go add a table of directives where we could also place info like this.

maxandersen avatar Jun 18 '25 06:06 maxandersen

@all-contributors please add @gunnarmorling for docs

maxandersen avatar Jun 18 '25 06:06 maxandersen

@maxandersen

I've put up a pull request to add @gunnarmorling! :tada:

allcontributors[bot] avatar Jun 18 '25 06:06 allcontributors[bot]