Skip to content

use more environment variables so the scripts are more portable#54

Merged
goodmami merged 2 commits intomainfrom
51-it-would-be-good-if-the-scripts-were-made-useful-for-other-projects-like-tufs-and-arasaac
Apr 22, 2025
Merged

use more environment variables so the scripts are more portable#54
goodmami merged 2 commits intomainfrom
51-it-would-be-good-if-the-scripts-were-made-useful-for-other-projects-like-tufs-and-arasaac

Conversation

@fcbond
Copy link
Copy Markdown
Contributor

@fcbond fcbond commented Mar 2, 2025

This allows me to use package, validate and summarize with tufs and arasaac.

@goodmami
Copy link
Copy Markdown
Collaborator

goodmami commented Mar 2, 2025

Repeating what we discussed: it's possible to set environment variables in the Google Actions scripts, but it may be easier to just make command-line options.

@goodmami
Copy link
Copy Markdown
Collaborator

@fcbond instead of command-line options, I just added back the variables with default values. If the environment variables are set, they will be used, otherwise they will work as before. Is this alright?

@goodmami goodmami merged commit 8e37e8a into main Apr 22, 2025
@goodmami
Copy link
Copy Markdown
Collaborator

Is this alright?

Assuming it is.

@goodmami goodmami deleted the 51-it-would-be-good-if-the-scripts-were-made-useful-for-other-projects-like-tufs-and-arasaac branch May 22, 2025 20:06
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.

It would be good if the scripts were made useful for other projects (like TUFS and ARASAAC)

2 participants