Tidy up logging

This commit is contained in:
Ana Hobden 2023-07-12 10:52:09 -07:00
parent 339fb7942a
commit 3bae1cc12f
2 changed files with 16 additions and 14 deletions

14
dist/index.js vendored
View file

@ -175,7 +175,7 @@ class NixInstallerAction {
execute_install(binary_path) { execute_install(binary_path) {
return __awaiter(this, void 0, void 0, function* () { return __awaiter(this, void 0, void 0, function* () {
const execution_env = this.executionEnvironment(); const execution_env = this.executionEnvironment();
actions_core.info(`Execution environment: ${JSON.stringify(execution_env)}`); actions_core.info(`Execution environment: ${JSON.stringify(execution_env, null, 4)}`);
const args = ['install']; const args = ['install'];
if (this.planner) { if (this.planner) {
args.push(this.planner); args.push(this.planner);
@ -192,10 +192,10 @@ class NixInstallerAction {
env: merged_env env: merged_env
}); });
spawned.stdout.on('data', data => { spawned.stdout.on('data', data => {
actions_core.info(`stdout: ${data}`); actions_core.info(data);
}); });
spawned.stderr.on('data', data => { spawned.stderr.on('data', data => {
actions_core.info(`stderr: ${data}`); actions_core.info(data);
}); });
const exit_code = yield new Promise((resolve, _reject) => { const exit_code = yield new Promise((resolve, _reject) => {
spawned.on('close', resolve); spawned.on('close', resolve);
@ -213,7 +213,7 @@ class NixInstallerAction {
if (this.reinstall) { if (this.reinstall) {
// We need to uninstall, then reinstall // We need to uninstall, then reinstall
actions_core.info('Nix was already installed, `reinstall` is set, uninstalling for a reinstall'); actions_core.info('Nix was already installed, `reinstall` is set, uninstalling for a reinstall');
yield this.uninstall(); yield this.execute_uninstall();
} }
else { else {
// We're already installed, and not reinstalling, just set GITHUB_PATH and finish early // We're already installed, and not reinstalling, just set GITHUB_PATH and finish early
@ -232,17 +232,17 @@ class NixInstallerAction {
actions_core.addPath('/nix/var/nix/profiles/default/bin'); actions_core.addPath('/nix/var/nix/profiles/default/bin');
actions_core.addPath(`${process.env.HOME}/.nix-profile/bin`); actions_core.addPath(`${process.env.HOME}/.nix-profile/bin`);
} }
uninstall() { execute_uninstall() {
return __awaiter(this, void 0, void 0, function* () { return __awaiter(this, void 0, void 0, function* () {
const spawned = (0, node_child_process_1.spawn)(`/nix/nix-installer`, ['uninstall'], { const spawned = (0, node_child_process_1.spawn)(`/nix/nix-installer`, ['uninstall'], {
env: Object.assign({ NIX_INSTALLER_NO_CONFIRM: 'true' }, process.env // To get $PATH, etc env: Object.assign({ NIX_INSTALLER_NO_CONFIRM: 'true' }, process.env // To get $PATH, etc
) )
}); });
spawned.stdout.on('data', data => { spawned.stdout.on('data', data => {
actions_core.info(`stdout: ${data}`); actions_core.info(data);
}); });
spawned.stderr.on('data', data => { spawned.stderr.on('data', data => {
actions_core.info(`stderr: ${data}`); actions_core.info(data);
}); });
const exit_code = yield new Promise((resolve, _reject) => { const exit_code = yield new Promise((resolve, _reject) => {
spawned.on('close', resolve); spawned.on('close', resolve);

View file

@ -193,7 +193,9 @@ class NixInstallerAction {
private async execute_install(binary_path: string): Promise<number> { private async execute_install(binary_path: string): Promise<number> {
const execution_env = this.executionEnvironment() const execution_env = this.executionEnvironment()
actions_core.info(`Execution environment: ${JSON.stringify(execution_env)}`) actions_core.info(
`Execution environment: ${JSON.stringify(execution_env, null, 4)}`
)
const args = ['install'] const args = ['install']
if (this.planner) { if (this.planner) {
@ -217,11 +219,11 @@ class NixInstallerAction {
}) })
spawned.stdout.on('data', data => { spawned.stdout.on('data', data => {
actions_core.info(`stdout: ${data}`) actions_core.info(data)
}) })
spawned.stderr.on('data', data => { spawned.stderr.on('data', data => {
actions_core.info(`stderr: ${data}`) actions_core.info(data)
}) })
const exit_code: number = await new Promise((resolve, _reject) => { const exit_code: number = await new Promise((resolve, _reject) => {
@ -243,7 +245,7 @@ class NixInstallerAction {
actions_core.info( actions_core.info(
'Nix was already installed, `reinstall` is set, uninstalling for a reinstall' 'Nix was already installed, `reinstall` is set, uninstalling for a reinstall'
) )
await this.uninstall() await this.execute_uninstall()
} else { } else {
// We're already installed, and not reinstalling, just set GITHUB_PATH and finish early // We're already installed, and not reinstalling, just set GITHUB_PATH and finish early
this.set_github_path() this.set_github_path()
@ -262,7 +264,7 @@ class NixInstallerAction {
actions_core.addPath(`${process.env.HOME}/.nix-profile/bin`) actions_core.addPath(`${process.env.HOME}/.nix-profile/bin`)
} }
async uninstall(): Promise<number> { async execute_uninstall(): Promise<number> {
const spawned = spawn(`/nix/nix-installer`, ['uninstall'], { const spawned = spawn(`/nix/nix-installer`, ['uninstall'], {
env: { env: {
NIX_INSTALLER_NO_CONFIRM: 'true', NIX_INSTALLER_NO_CONFIRM: 'true',
@ -271,11 +273,11 @@ class NixInstallerAction {
}) })
spawned.stdout.on('data', data => { spawned.stdout.on('data', data => {
actions_core.info(`stdout: ${data}`) actions_core.info(data)
}) })
spawned.stderr.on('data', data => { spawned.stderr.on('data', data => {
actions_core.info(`stderr: ${data}`) actions_core.info(data)
}) })
const exit_code: number = await new Promise((resolve, _reject) => { const exit_code: number = await new Promise((resolve, _reject) => {