Skip to content

Commit 329668f

Browse files
Merge #2251
2251: Adding setup-as prior to npm publish. r=sacherjj a=sacherjj Due to changes in makefile, setup-as is no longer run as part of make deb. This explicitly adds a drone step to run this prior to publish. Co-authored-by: Joe Sacher <[email protected]>
2 parents 462aa9a + a705f2e commit 329668f

2 files changed

Lines changed: 5 additions & 1 deletion

File tree

.drone.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -569,6 +569,11 @@ steps:
569569
CARGO_TOKEN:
570570
from_secret: crates_io_token
571571

572+
- name: as-setup
573+
image: casperlabs/node-build-u1804
574+
commands:
575+
- "make setup-as"
576+
572577
- name: as-contract-publish
573578
image: plugins/npm
574579
settings:

node/src/reactor.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -672,7 +672,6 @@ where
672672
};
673673
if let Err(error) = self.scheduler.debug_dump(&mut file).await {
674674
warn!(%error, "could not serialize debug snapshot to {}", debug_dump_filename);
675-
return;
676675
}
677676
}
678677

0 commit comments

Comments
 (0)