Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Python setup and fix multiline prerequisites output #3

Merged
merged 3 commits into from
Sep 7, 2024
Merged

Conversation

colluca
Copy link
Owner

@colluca colluca commented Sep 7, 2024

  • Extend test to generate a list with multiple prerequisites.
  • Fix multiline prerequisite output, by converting newlines to spaces. This is in general more useful, as this file list would be used as an argument to a command-line tool.
  • Remove Python setup to avoid conflicts with Python installations on e.g. workflows using a Docker container with a preinstalled Python toolchain.
  • Improve pip invocation and explicitly specify path to requirements.txt file in action's repository.
  • Sort prerequisites to ensure consistency

@colluca colluca marked this pull request as ready for review September 7, 2024 09:48
@colluca colluca merged commit 08ef974 into main Sep 7, 2024
2 checks passed
@colluca colluca deleted the fix branch September 7, 2024 09:55
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