Skip to content

feat(#838): disable linting with --blind flag to improve performance#850

Merged
yegor256 merged 2 commits intoobjectionary:masterfrom
volodya-lombrozo:838-make-lints-optional
Feb 18, 2026
Merged

feat(#838): disable linting with --blind flag to improve performance#850
yegor256 merged 2 commits intoobjectionary:masterfrom
volodya-lombrozo:838-make-lints-optional

Conversation

@volodya-lombrozo
Copy link
Copy Markdown
Member

This PR makes the lint step ignored when the --blind flag is provided, improving command execution speed.

Fixes #838

@volodya-lombrozo
Copy link
Copy Markdown
Member Author

@yegor256 we can disable listing with the use of --blind option. No need to touch --easy option.

Comment thread README.md Outdated

## Options

You can apply the following options to the `eoc` command:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@volodya-lombrozo this looks like a duplication of information to me. What happens when the actual list of options of eoc changes and we forget to update the README file?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yegor256 should we remove Commands section for consistency?

https://github.com/objectionary/eoc?tab=readme-ov-file#commands

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yegor256 we can think how to generate these docs automatically in a separate PR. What do you think?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@volodya-lombrozo yes, indeed, this would be helpful

@volodya-lombrozo
Copy link
Copy Markdown
Member Author

volodya-lombrozo commented Feb 18, 2026

@yegor256 I have created a task for it:

#857

(and removed the Options section)

@yegor256 yegor256 merged commit 54879d2 into objectionary:master Feb 18, 2026
16 checks passed
@0crat
Copy link
Copy Markdown

0crat commented Feb 18, 2026

@volodya-lombrozo Thanks for the contribution! You've earned +16 points for providing code reviews, which aligns perfectly with our policy that rewards collaborative development. Your contribution demonstrates the quality-focused approach we value - keep them coming! Your running score is +495 and growing; don't forget to check your Zerocracy account for the latest updates. 🚀

@0crat
Copy link
Copy Markdown

0crat commented Mar 2, 2026

@yegor256 Thanks for the review! You've earned +7 points for this contribution: +12 as the base reward for conducting a code review, then -5 points deducted due to insufficient reviewer engagement (only 1 comment during the review, which is below our 12-comment threshold). Your running score is +18; don't forget to check your Zerocracy account too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

make lints optional

3 participants