Skip to content

Conversation

@bbrala
Copy link
Contributor

@bbrala bbrala commented Nov 28, 2025

This pull request enhances the handling of remote variable files by allowing multiple --remote-variables flags, enabling users to fetch and merge several variable files from different remote Git repositories. The changes update both the CLI interface and the internal logic to support arrays of remote variable sources and ensure the merged variables are correctly processed.

Support for multiple remote variable files:

  • The --remote-variables CLI option now accepts multiple values (as an array), allowing users to specify more than one remote variable file. (src/index.ts, src/argv.ts, README.md) [1] [2] [3]
  • The logic in VariablesFromFiles iterates over each provided remote variable source, fetches the variable files, and merges them using deepmerge to combine all variables into a single object. (src/variables-from-files.ts) [1] [2]

Testing and documentation:

  • The test for remote variables was updated to use an array for remoteVariables, ensuring the new behavior is covered. (tests/test-cases/remote-variables-file/integration.test.ts)
  • Documentation in README.md was updated to explain how to use multiple --remote-variables flags.

(the code was not generated with AI ;))

@bbrala bbrala force-pushed the feature/remote-variabled-array branch from 64c3514 to ea080e1 Compare November 28, 2025 12:52
bbrala and others added 2 commits November 29, 2025 10:59
@bbrala
Copy link
Contributor Author

bbrala commented Nov 29, 2025

Updated the PR to use the proper library, did need some extra checks around the return value of the yaml library since it is not very specific.

@firecow firecow merged commit d0582ac into firecow:master Dec 1, 2025
9 checks passed
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.

2 participants