--chaos should warn instead of error on recipe lint checks #497

New Issue

3wordchant · 2023-10-03T20:20:04Z

3wordchant commented

2023-10-03 20:20:04 +00:00

Deep in the hacking mines - it feels like the intention of specifying --chaos should turn recipe lint errors into warnings

Deep in the hacking mines - it feels like the intention of specifying `--chaos` should turn recipe lint errors into warnings

3wordchant added the

bug

label 2023-10-03 20:20:04 +00:00

decentral1se commented

2023-10-03 23:54:57 +00:00

@3wordchant True!

The LintForErrors logic errors out on first linting error, would you like to see all errors turned into warnings and outputted? That might be a good change to make in general, to see all possible errors/warnings at once instead of running into them one by one?

Altho, probably the simplest thing to do on the code side would be to warn and skip linting altogether when --chaos. Unless you think it's handy & worth it to see your potential errors while --chaos'in ❔

@3wordchant True! The `LintForErrors` logic errors out on first linting error, would you like to see all errors turned into warnings and outputted? That might be a good change to make in general, to see all possible errors/warnings at once instead of running into them one by one? Altho, probably the simplest thing to do on the code side would be to warn and skip linting altogether when `--chaos`. Unless you think it's handy & worth it to see your potential errors while `--chaos`'in ❔

decentral1se added the

abra

label 2024-03-27 06:24:34 +00:00

Sign in to join this conversation.

No Milestone

No project

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: toolshed/organising#497