Skip to content

Commit

Permalink
Use merge to add build results to publish context
Browse files Browse the repository at this point in the history
  • Loading branch information
dappnodedev committed Jul 16, 2024
1 parent 3416e78 commit 557c676
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 22 deletions.
23 changes: 4 additions & 19 deletions src/tasks/publish/subtasks/getBuildAndUploadTask.ts
Original file line number Diff line number Diff line change
@@ -1,8 +1,9 @@
import Listr, { ListrTask } from "listr";
import { BuildAndUploadOptions } from "../../buildAndUpload/types.js";
import { ListrContextBuild, ListrContextPublish } from "../../../types.js";
import { ListrContextPublish } from "../../../types.js";
import { buildAndUpload } from "../../buildAndUpload/index.js";
import { VerbosityOptions } from "../../../commands/build/types.js";
import { merge } from "lodash-es";

export function getBuildAndUploadTask({
buildOptions,
Expand All @@ -21,24 +22,8 @@ export function getBuildAndUploadTask({

const buildResults = await buildTasks.run();

copyBuildCtxToPublishCtx({
buildCtx: buildResults,
publishCtx: ctx
});
// Add build results to publish context
ctx = merge(ctx, buildResults);
}
};
}

function copyBuildCtxToPublishCtx({
buildCtx,
publishCtx
}: {
buildCtx: ListrContextBuild;
publishCtx: ListrContextPublish;
}) {
for (const [key, result] of Object.entries(buildCtx)) {
publishCtx[key] = publishCtx[key]
? { ...publishCtx[key], ...result }
: result;
}
}
10 changes: 7 additions & 3 deletions src/tasks/publish/subtasks/getFetchApmVersionsTask.ts
Original file line number Diff line number Diff line change
Expand Up @@ -16,16 +16,20 @@ export function getFetchNextVersionsFromApmTask({
title: "Fetch current versions from APM",
task: async ctx => {

for (const { manifest } of packagesToBuildProps) {
for (const { manifest, variant, releaseDir } of packagesToBuildProps) {
const dnpName = manifest.name;

ctx[dnpName] = ctx[dnpName] || {};
ctx[dnpName].nextVersion = await getNextPackageVersion({
const nextVersion = await getNextPackageVersion({
manifest,
releaseType,
ethProvider
});

ctx[dnpName] = {
variant,
releaseDir,
nextVersion
}
}
}
};
Expand Down

0 comments on commit 557c676

Please sign in to comment.