Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add default: for connection_info attributes on transports #231

Open
DavidS opened this issue Sep 30, 2019 · 0 comments
Open

Add default: for connection_info attributes on transports #231

DavidS opened this issue Sep 30, 2019 · 0 comments
Labels
enhancement

Comments

@DavidS
Copy link
Contributor

DavidS commented Sep 30, 2019

Use Case

Optional attributes for transport connection_info often have default values. Currently this requires calling out the default value in the desc string and defining a fallback calculation in code, leading to duplication at best and inconsistency at worst.

Describe the Solution You Would Like

Define a default: key equivalent to the attributes in types.
seanmil added a commit to seanmil/puppet-resource_api that referenced this issue Dec 20, 2021
@seanmil
(puppetlabsGH-231) Add default to transport attributes
01ad036 
Add the ability to specify a default to transport connection_info
attributes which will be used in the event no value for the given
connection_info attribute is provided.
seanmil added a commit to seanmil/puppet-resource_api that referenced this issue Jan 12, 2022
@seanmil
(puppetlabsGH-231) Add default to transport attributes
2a223cd 
Add the ability to specify a default to transport connection_info
attributes which will be used in the event no value for the given
connection_info attribute is provided.
michaeltlombardi pushed a commit to seanmil/puppet-resource_api that referenced this issue Jan 24, 2022
@seanmil @michaeltlombardi
(puppetlabsGH-231) Add default to transport attributes
aa2b34c 
Add the ability to specify a default to transport connection_info
attributes which will be used in the event no value for the given
connection_info attribute is provided.
michaeltlombardi added a commit that referenced this issue Jan 24, 2022
@michaeltlombardi
Merge pull request #293 from seanmil/default_connection_params
4dd711f 
(GH-231) Add default to transport attributes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@DavidS