Skip to content

Comments

No subshell#15

Open
fiveop wants to merge 4 commits intocarpentries:mainfrom
fiveop:no_subshell
Open

No subshell#15
fiveop wants to merge 4 commits intocarpentries:mainfrom
fiveop:no_subshell

Conversation

@fiveop
Copy link

@fiveop fiveop commented Nov 28, 2024

The strings for two error messages contained syntax to open subshells, as the author used markdown syntax for inline code spans without escaping the backquotes.

The second commit fixes a typo in a package name.

Pushed two more commits:

  • changed argument handling
  • removed trailing whitespace

- check for minimum amount of arguments
- exit if this check fails
- allow for multiple arguments passed to jinja2
- quote filenames to allow for spaces in filenames
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