gagb
gagb
@r3d91ll before I review your PR what do you think about: https://github.com/microsoft/autogen/issues/1286
> > @r3d91ll before I review your PR what do you think about: #1286 > > I'm in full support of it. I didn't have the time to track down...
> > > > @r3d91ll before I review your PR what do you think about: #1286 > > > > > > > > > I'm in full support of...
> > > > > @r3d91ll before I review your PR what do you think about: #1286 > > > > > > > > > > > > I'm...
@r3d91ll the other PR was merged, please feel free to suggest changes per your this PR: https://github.com/microsoft/autogen/pull/1333
> > @r3d91ll before I review your PR what do you think about: #1286 > > I'm in full support of it. I didn't have the time to track down...
@r3d91ll I am closing this PR. Please re-open if you need these changes.
I just tried putting the `samples/dockers/Dockerfile.base` in `.devcontainer` with the following settings and it worked with Codespaces -- I was able to rebuild the container. I think unification is possible....
I suggest the following solution: create subdir for each dockerfile in .devcontainer. This way the docker files will be compatible with both Codespaces and non-codespaces environment while minimizing duplicate locations...
> > I suggest the following solution: > > create subdir for each dockerfile in .devcontainer. This way the docker files will be compatible with both Codespaces and non-codespaces environment...