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

Fix apply all with project name bug #565

Merged
merged 1 commit into from
Apr 2, 2019
Merged

Commits on Apr 2, 2019

  1. Fix apply all with project name bug

    Previously, if you were using project names and you had two projects
    with the same directory and workspace but different workflows, running
    "atlantis apply" would fail. This was because we weren't figuring out
    what the project name was for each pending plan.
    
    This change fixes this by extracting the project name from the planfile.
    We also handle /'s in the project name by substituting them with '::'.
    This allows us to convert them back to /'s when we're figuring out the
    project names from the filenames. It's safe to do this because : isn't
    an allowed character for project names.
    
    We also remove a final substitution of invalid filename characters
    because we rely on the validation of the project names instead which
    happens when validating the atlantis.yaml file.
    lkysow committed Apr 2, 2019
    Configuration menu
    Copy the full SHA
    609bf84 View commit details
    Browse the repository at this point in the history