Skip to content

Conversation

@julian-smith-artifex-com
Copy link
Contributor

@julian-smith-artifex-com julian-smith-artifex-com commented Nov 28, 2025

This allows arguments to be read from a file with @PATH.

It uses https://docs.python.org/3/library/argparse.html#fromfile-prefix-chars.

I'd like this change specifically to avoid having to specify a large number of paths on the command line - when codespell reports errors, ptest dumps the entire command line which makes it difficult to find the actual codespell output.

There is currently one test failure, of codecov/patch. I don't understand what's going on here - the new function convert_arg_line_to_args() is definitely called by new test test_args_from_file().

@autofix-ci
Copy link
Contributor

autofix-ci bot commented Nov 28, 2025

Hi! I'm autofix logoautofix.ci, a bot that automatically fixes trivial issues such as code formatting in pull requests.

I would like to apply some automated changes to this pull request, but it looks like I don't have the necessary permissions to do so. To get this pull request into a mergeable state, please do one of the following two things:

  1. Allow edits by maintainers for your pull request, and then re-trigger CI (for example by pushing a new commit).
  2. Manually fix the issues identified for your pull request (see the GitHub Actions output for details on what I would like to change).

@julian-smith-artifex-com
Copy link
Contributor Author

I'm not sure why the codecov/patch test is failing - the new test definitely ends up using the new convert_arg_line_to_args() function.

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.

1 participant