chalice icon indicating copy to clipboard operation
chalice copied to clipboard

Follow symlinks in chalicelib (resolves #696)

Open jessebrennan opened this issue 3 years ago • 7 comments

Resolves issue #696

Follow symlinks when scanning chalicelib directory.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

jessebrennan avatar Jan 24 '22 23:01 jessebrennan

Looks good, I'd just like to add a test for this in tests/functinal/test_deployer.py to make sure we don't regress on this functionality.

jamesls avatar Jan 25 '22 18:01 jamesls

@jamesls I wasn't sure from your comment if you planned to do it or were asking me to, but I went ahead and added the test.

I tried to push commits one at a time so that you could see the CI failure with just the test, then pass with the fix, but it looks like they won't run without approval.

Anyway thanks for considering this.

jessebrennan avatar Jan 25 '22 20:01 jessebrennan

@jamesls I thought I'd ping you again in case this slipped through the cracks.

jessebrennan avatar Feb 08 '22 23:02 jessebrennan

@jamesls I wanted to ping about this PR before it goes stale. Please let me know if you're no longer interested in merging. I added the requested tests.

jessebrennan avatar Jul 14 '22 15:07 jessebrennan

Agree, it would be nice to get this in

mwoodsmall avatar Jul 29 '22 10:07 mwoodsmall

@jamesls is there any interest in merging this still?

jessebrennan avatar Oct 03 '22 20:10 jessebrennan

@jamesls Update on this?

philtrep avatar Jun 21 '23 05:06 philtrep