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.

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

validate class and defined resource parameters #81

Merged
merged 1 commit into from Aug 19, 2021

Conversation

nod0n
Copy link
Contributor

@nod0n nod0n commented Jul 30, 2021

Pull Request (PR) description

This introduces two new type aliases: file_name and user_name

Validating parameters simplifies debugging and improves stability as
problematic parameters can be prohibited.

@nod0n nod0n marked this pull request as draft August 18, 2021 08:14
@nod0n nod0n force-pushed the parameter_validation branch 3 times, most recently from a324d33 to 551ce2e Compare August 19, 2021 17:52
@nod0n nod0n marked this pull request as ready for review August 19, 2021 17:56
types/file_name.pp Outdated Show resolved Hide resolved
types/user_name.pp Outdated Show resolved Hide resolved
@nod0n nod0n changed the title Parameter validation validate class and define parameters Aug 19, 2021
This itroduces two new type aliases: file_name and user_name

Validating parameters simplifies debugging and improves stability as
problematic parameters can be prohibited.
@bastelfreak bastelfreak changed the title validate class and define parameters validate class and defined resource parameters Aug 19, 2021
@bastelfreak bastelfreak added the enhancement New feature or request label Aug 19, 2021
@bastelfreak bastelfreak merged commit 9a39bbf into voxpupuli:master Aug 19, 2021
@nod0n nod0n deleted the parameter_validation branch August 19, 2021 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants