Why can't I use validations on environment_variables in bashly.yml

[ichoosetoaccept]

Bashly supports validations for arguments and flag arguments but not for environment variables. Why?
For example, this is throwing errors on bashly generate:

environment_variables: - name: RELEASES_FOLDER help: Path to the releases folder default: /export/home/releases validate: dir_exists - name: LOG_DIR help: Directory for storing log files default: ${RELEASES_FOLDER}/logs validate: dir_exists 

This would save me from having to write my own "folder exists" validation function.. unless I'm missing something?

Thanks for reading!

[DannyBen]

I need to look into it. I don't remember if the omission was intentional or not.

[ichoosetoaccept]

I worked around it by doing this:
bashly add validations
Then in initialize.sh:

# Check if RELEASES_FOLDER exists validate_dir_exists "$RELEASES_FOLDER" 

Seems a bit hacky but I guess it does work :)

[DannyBen]

Hacky. I have implemented it in #546.
Initial implementation works, I just need now to complete tests, docs etc.

Will be available soon.

One thing it raises - the custom strings file is showing its age. I will need to normalize it (read: rename some keys) - but this will be a breaking change for a future release.

[ichoosetoaccept]

Awesome, thank you!

[DannyBen]

This is now merged to master, thanks for the report.
It will be released either this weekend or next weekend. In the meantime, you can use the edge version.

If you happen to test it, it would be great if you can report that it works as expected (or more importantly, that it doesn't).

[ichoosetoaccept]

Will gladly report back as soon as I have a chance to test. Thanks for the crazy fast turnaround time!

[DannyBen]

My pleasure.