Skip to content

Add inputs for regex, files, after_context, before_context, context#86

Open
spetrosi wants to merge 5 commits into
codespell-project:masterfrom
spetrosi:add-regex-input
Open

Add inputs for regex, files, after_context, before_context, context#86
spetrosi wants to merge 5 commits into
codespell-project:masterfrom
spetrosi:add-regex-input

Conversation

@spetrosi

@spetrosi spetrosi commented Feb 5, 2025

Copy link
Copy Markdown

Adding some missing parameters from CLI and config file to GH action.

@spetrosi spetrosi changed the title Add regex input Add regex and files inputs Feb 5, 2025

@peternewman peternewman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Some copy/paste errors to address...

Comment thread entrypoint.sh Outdated
Comment thread entrypoint.sh Outdated
Comment thread entrypoint.sh
fi
echo "Files or directories to check: '${INPUT_SKIP}'"
if [ "x${INPUT_FILES}" != "x" ]; then
command_args="${command_args} --regex ${INPUT_FILES}"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Which argument are you trying to populate here?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

[spetrosi@fedora network]$ codespell --help | grep "\--regex" -A5
  -r REGEX, --regex REGEX
                        regular expression that is used to find words. By
                        default any alphanumeric character, the underscore,
                        the hyphen, and the apostrophe are used to build
                        words. This option cannot be specified together with
                        --write-changes.

@spetrosi spetrosi changed the title Add regex and files inputs Add inputs for regex, files, after_context, before_context, context Feb 6, 2025
@spetrosi spetrosi requested a review from peternewman February 7, 2025 11:54
@spetrosi

Copy link
Copy Markdown
Author

@peternewman any update on this?

@larsoner

larsoner commented Nov 5, 2025

Copy link
Copy Markdown
Member

@spetrosi can you fix the conflicts?

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.

3 participants