FAQ | This is a LIVE service | Changelog

Skip to content
Snippets Groups Projects

feat: enable setting environment variables on the function

Merged Dr Rich Wareham requested to merge enable-environment-variables into master

In most cases, it is more convenient to set non-secret information via environment variables. Expose this functionality.

This MR also contains a few small fixes:

  • The syntax in outputs.tf was incorrect if a custom service account was supplied. In this case the google_service_account.script tuple would be empty and google_service_account.script[0] would error. Move to using the ?! operator which shortcuts its arguments and so never evaluates the troublesome expression if a custom service account is provided.
  • The pre-commit hook markdownlint-cli2 is currently failing to initialise itself and download the nodejs runtime. Comment it out for the moment.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading