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

path_relative_to_include() helper should be OS agnostic #99

Closed
brikis98 opened this issue Jan 12, 2017 · 1 comment
Closed

path_relative_to_include() helper should be OS agnostic #99

brikis98 opened this issue Jan 12, 2017 · 1 comment
Labels
bug Something isn't working

Comments

@brikis98
Copy link
Member

The path_relative_to_include() returns the path between a root and a child .terragrunt file. This is typically used to set the state_file_id and key settings. The problem is that on *nix systems, this path will contain forward slashes, whereas on Windows, for the same path, it will contain backslashes. Therefore, the locking and state file storage won't work properly if team members use different operating systems.

path_relative_to_include should probably enforce forward slashes on all OS's.

@brikis98
Copy link
Member Author

This was fixed by #100.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant