Skip to content

Conversation

DannyBen
Copy link
Member

@DannyBen DannyBen commented Nov 13, 2023

As reported in #445 - it appears that some rendering views are incompatible with set -u strict mode, and cause an unbound variable in the rendered script. This PR fixes it and improves tests to cover this use case.

Notable changes:

  1. Some views and libraries were changed to avoid unbound variable by changing $arg['something'] to ${arg['something']:-} (or a similar syntax as appropriate).
  2. The examples spec was changed to enable strict mode before running to ensure all examples are generated in strict mode and pass tests.

@DannyBen DannyBen merged commit b0b979c into master Nov 13, 2023
@DannyBen DannyBen deleted the fix/strict-mode branch November 13, 2023 07:57
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