Main Goals

• Simplify root directory so the repository feels more accessible to first time readers or contributors.
  • There are 4 sample* directories and 2 reproduction* directories that should be more clearly identified as test projects
  • There are several build and test files cluttered in the root directory that could be placed in more specific subdirectories
• Change the convention of "customer-samples" to "samples". My assumption is that new developers would expect to look at samples as small code projects that use the provided SDK in some meaningful way, so this proposed change would align with that.
• Change the convention of "samples" to "test-applications". The "samples" in our repository really act as test applications to ensure our automatic instrumentation works as advertised. Renaming to "test-applications" is a little more clear about the intent, but I'm open to other suggestions on the name of this.
• Change the convention of "reproductions" to "regression". These test applications serve as small regression tests to make sure previous bugs do not reoccur. Renaming to regression might convey that idea more clearly.

Proposed directory structure

Directory: /

Directory: /build

Directory: /blog

Contents unchanged

Directory: /docs

Contents unchanged

Directory: /samples (renamed from /customer-samples)

Contents unchanged

Directory: /src

Directory: /test

Directory: /tools

Directory: /test/test-applications

Updates

• Originally the two subfolders aspnet and iis were proposed as subdirectories of /test/Datadog.Trace.ClrProfiler.IntegrationTests/test-applications. Instead, they'll be combined into the aspnet subdirectory
• Originally the test-applications directory was proposed to be a subdirectory of /test/Datadog.Trace.ClrProfiler.IntegrationTests. Instead, it will be a subdirectory of test
• Updated new directory name to /src/WindowsInstaller based on feedback
• Updated the proposal to reflect the directory move from /lib to /src/Datadog.Trace.ClrProfiler.Native/lib