Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@adasauce
Copy link

added this feature in because it integrates a bit better with my deployment story.
apologies for the drive by PR if it's not necessary or desired.

Copy link
Owner

@F1bonacc1 F1bonacc1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That looks great!
Thanks, @adasauce!
One minor request, can you please update the documentation to let the users know that this is now supported? Here:

#### Rendered Parameters:
* `processes.process.command`
* `processes.process.working_dir`
* `processes.process.log_location`
* `processes.process.description`

@adasauce adasauce force-pushed the env-var-rendering branch from 806aa39 to 031be4f Compare July 16, 2025 21:51
Copy link
Owner

@F1bonacc1 F1bonacc1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!
Once the conflicts are resolved, ready to merge.

@adasauce adasauce force-pushed the env-var-rendering branch from 031be4f to 7661f9e Compare July 16, 2025 21:52
@adasauce adasauce force-pushed the env-var-rendering branch from 7661f9e to 17741f8 Compare July 16, 2025 21:53
@sonarqubecloud
Copy link

@F1bonacc1 F1bonacc1 merged commit 4ce7420 into F1bonacc1:main Jul 16, 2025
6 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