Restructure repo layout to clarify feature vertical boundaries #4132
Conversation
|
/runUiTests |
Conflicts: plugins/toolkit/jetbrains-core/src/software/aws/toolkits/jetbrains/services/codemodernizer/ideMaven/MavenRunnerUtils.kt plugins/toolkit/jetbrains-core/src/software/aws/toolkits/jetbrains/services/codemodernizer/model/MavenCopyCommandsResult.kt
rli
changed the title
| afterSync(":sdk-codegen:generateSdks") | ||
| afterSync(":jetbrains-core:generateTelemetry") | ||
| afterSync(":plugin-core:sdk-codegen:generateSdks") | ||
| afterSync(":plugin-toolkit:jetbrains-core:generateTelemetry") |
There was a problem hiding this comment.
Should this also be a part of plugin-core?
There was a problem hiding this comment.
that is more involved so will happen later
There was a problem hiding this comment.
It's an empty file, is this required?
There was a problem hiding this comment.
i believe the path needs to exist you can't commit an empty folder
this should probably handled a different way but is not in scope for this PR
Conflicts: plugins/toolkit/jetbrains-core/src/software/aws/toolkits/jetbrains/services/cwc/controller/chat/telemetry/QTelemetryUtils.kt
| jetbrains-core/bin/ | ||
| jetbrains-ultimate/bin/ | ||
| resources/bin/ | ||
| jetbrains-rider/**/NuGet.config |
There was a problem hiding this comment.
Will these path names need to be changed?
There was a problem hiding this comment.
we can adjust again if we change the paths again, but it shouldn't be necessary
|
Along the lines of aws/aws-toolkit-vscode#4342,
We split the structure of the repo to prepare for multiple extensions out of a single monorepo.
This is only a subset of changes to hopefully keep merge conflicts minimal as we migrate buildscripts and further move code between directories.
License
I confirm that my contribution is made under the terms of the Apache 2.0 license.