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

config: smarter provider alias usage validation #10807

Merged
merged 1 commit into from
Dec 19, 2016
Merged

Commits on Dec 17, 2016

  1. config: smarter provider alias usage validation

    Fixes #4789
    
    This improves the validation that valid provider aliases are used.
    
    Previously, we required that provider aliases be defined in every module
    they're used. This isn't correct because the alias may be used in a
    parent module and inherited.
    
    This removes that validation and creates the validation that a provider
    alias must be defined in the used module or _any parent_. This allows
    inheritance to work properly.
    
    We've always had this type of validation for aliases because we believe
    its a good UX tradeoff: typo-ing an alias is really painful, so we
    require declaration of alias usage. It may add a small burden to
    declare, but since relatively few aliases are used, it improves the
    scenario where a user fat-fingers an alias name.
    mitchellh committed Dec 17, 2016
    Configuration menu
    Copy the full SHA
    0c30cae View commit details
    Browse the repository at this point in the history