Skip to content

Commit f2c27fa

Browse files
authored
Merge c42f240 into f665499
2 parents f665499 + c42f240 commit f2c27fa

File tree

4 files changed

+17
-31
lines changed

4 files changed

+17
-31
lines changed

CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,8 @@ For experimental package changes, see the [experimental CHANGELOG](experimental/
1515

1616
### :house: (Internal)
1717

18+
* chore: no need for 'packages' in lerna.json [#4264](https://github.com/open-telemetry/opentelemetry-js/pull/4264) @trentm
19+
1820
## 1.18.1
1921

2022
### :bug: (Bug Fix)

lerna.json

Lines changed: 1 addition & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,5 @@
11
{
22
"version": "independent",
33
"npmClient": "npm",
4-
"useWorkspaces": true,
5-
"// packages": "Please sync with package.json#workspaces",
6-
"packages": [
7-
"api",
8-
"packages/*",
9-
"experimental/packages/*",
10-
"experimental/examples/*",
11-
"experimental/backwards-compatibility/*",
12-
"integration-tests/*",
13-
"selenium-tests",
14-
"examples/otlp-exporter-node",
15-
"examples/opentelemetry-web",
16-
"examples/http",
17-
"examples/https",
18-
"examples/esm-http-ts"
19-
]
4+
"useWorkspaces": true
205
}

package.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,6 @@
102102
},
103103
"cacheDir": ".changelog"
104104
},
105-
"// workspaces": "Please sync with lerna.json#packages",
106105
"workspaces": [
107106
"api",
108107
"packages/*",

scripts/update-ts-configs.js

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -71,23 +71,23 @@ main();
7171
function main() {
7272
const pkgRoot = process.cwd();
7373
const projectRoot = findProjectRoot(pkgRoot);
74-
const lernaPackages = resolveLernaPackages(projectRoot);
74+
const workspacePackages = resolveWorkspacePackages(projectRoot);
7575

76-
generateTsConfig(projectRoot, lernaPackages, pkgRoot, true);
77-
for (const packageMeta of lernaPackages.values()) {
78-
generateTsConfig(projectRoot, lernaPackages, path.join(projectRoot, packageMeta.dir), false, packageMeta);
76+
generateTsConfig(projectRoot, workspacePackages, pkgRoot, true);
77+
for (const packageMeta of workspacePackages.values()) {
78+
generateTsConfig(projectRoot, workspacePackages, path.join(projectRoot, packageMeta.dir), false, packageMeta);
7979
}
8080
}
8181

82-
function generateTsConfig(projectRoot, lernaProjects, pkgRoot, isLernaRoot, packageMeta) {
82+
function generateTsConfig(projectRoot, workspacePackages, pkgRoot, isLernaRoot, packageMeta) {
8383
// Root tsconfig.json
8484
if (isLernaRoot) {
85-
writeRootTsConfigJson(pkgRoot, projectRoot, lernaProjects);
85+
writeRootTsConfigJson(pkgRoot, projectRoot, workspacePackages);
8686
return;
8787
}
8888

8989
const otelDependencies = getOtelDependencies(packageMeta.pkgJson);
90-
const dependenciesDir = resolveDependencyDirs(lernaProjects, otelDependencies);
90+
const dependenciesDir = resolveDependencyDirs(workspacePackages, otelDependencies);
9191
const references = dependenciesDir.map(it => path.relative(pkgRoot, path.join(projectRoot, it))).sort();
9292

9393
if (packageMeta.hasMultiTarget) {
@@ -97,24 +97,24 @@ function generateTsConfig(projectRoot, lernaProjects, pkgRoot, isLernaRoot, pack
9797
writeSingleTargetTsConfig(pkgRoot, projectRoot, references);
9898
}
9999

100-
function writeRootTsConfigJson(pkgRoot, projectRoot, lernaProjects) {
100+
function writeRootTsConfigJson(pkgRoot, projectRoot, workspacePackages) {
101101
const tsconfigPath = path.join(pkgRoot, 'tsconfig.json');
102102
const tsconfig = readJSON(tsconfigPath);
103-
const references = Array.from(lernaProjects.values())
103+
const references = Array.from(workspacePackages.values())
104104
.filter(it => it.isTsProject)
105105
.map(it => toPosix(path.relative(pkgRoot, path.join(projectRoot, it.dir)))).sort();
106106
tsconfig.references = references.map(path => {
107107
return { path: toPosix(path) }
108108
});
109-
tsconfig.typedocOptions.entryPoints = Array.from(lernaProjects.values())
109+
tsconfig.typedocOptions.entryPoints = Array.from(workspacePackages.values())
110110
.filter(it => !it.private && it.isTsProject)
111111
.map(it => toPosix(path.relative(pkgRoot, path.join(projectRoot, it.dir)))).sort();
112112
writeJSON(tsconfigPath, tsconfig, dryRun);
113113

114114
for (const tsconfigName of ['tsconfig.esm.json', 'tsconfig.esnext.json']) {
115115
const tsconfigPath = path.join(pkgRoot, tsconfigName);
116116
const tsconfig = readJSON(tsconfigPath);
117-
const references = Array.from(lernaProjects.values())
117+
const references = Array.from(workspacePackages.values())
118118
.filter(it => it.isTsProject && it.hasMultiTarget)
119119
.map(it => toPosix(path.relative(pkgRoot, path.join(projectRoot, it.dir)))).sort();
120120
tsconfig.references = references.map(pkgPath => {
@@ -182,10 +182,10 @@ function getOtelDependencies(packageJson) {
182182
return Array.from(deps.values());
183183
}
184184

185-
function resolveLernaPackages(projectRoot) {
185+
function resolveWorkspacePackages(projectRoot) {
186186
const map = new Map();
187-
const lernaJson = readJSON(`${projectRoot}/lerna.json`);
188-
for (const pkgDefinition of lernaJson.packages) {
187+
const packageJson = readJSON(`${projectRoot}/package.json`);
188+
for (const pkgDefinition of packageJson.workspaces) {
189189
if (ignoredLernaProjects.includes(pkgDefinition)) {
190190
continue;
191191
}

0 commit comments

Comments
 (0)