Skip to content

Commit a6b61e0

Browse files
committed
Got rid of setting action output to a logging message and then to the cpanm script...? I think the intent was to output information to the console, since there is no declared output variables in the action.yml. Also, setOutput is being removed, so if it needs to save output information, it should use the new environment file data.
1 parent efc3ecf commit a6b61e0

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

index.js

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -30,12 +30,12 @@ async function install_cpanm_location() {
3030
async function install_cpanm(install_to) {
3131
const url = "https://cpanmin.us";
3232

33-
core.setOutput(`Get cpanm from ${url}`);
33+
core.info(`Get cpanm from ${url}`);
3434

3535
const cpanmScript = await tc.downloadTool(url);
3636

37-
core.setOutput("cpanm", cpanmScript);
38-
console.log(`install_to ${install_to}`);
37+
core.info(`cpanm Script: ${cpanmScript}`);
38+
core.info(`install_to ${install_to}`);
3939

4040
const platform = os.platform();
4141

@@ -79,7 +79,7 @@ async function do_exec(cmd) {
7979
const platform = os.platform();
8080
const bin = sudo && platform != "win32" ? "sudo" : cmd.shift();
8181

82-
console.log(`do_exec: ${bin}`);
82+
core.info(`do_exec: ${bin}`);
8383

8484
await exec.exec(bin, cmd);
8585
}
@@ -126,7 +126,7 @@ async function run() {
126126
/* install one ore more modules */
127127
if (install !== null && install.length) {
128128
// install one or more modules
129-
console.log(`install: ${install}!`);
129+
core.info(`install: ${install}!`);
130130
const list = install.split("\n");
131131

132132
var cmd = [...CMD_install]; /* clone array */
@@ -139,9 +139,9 @@ async function run() {
139139
/* install from cpanfile */
140140
if (cpanfile !== null && cpanfile.length) {
141141
// install one or more modules
142-
console.log(`cpanfile: ${cpanfile}!`);
142+
core.info(`cpanfile: ${cpanfile}!`);
143143
const cpanfile_full_path = path.resolve(cpanfile);
144-
console.log(`cpanfile: ${cpanfile_full_path}! [resolved]`);
144+
core.info(`cpanfile: ${cpanfile_full_path}! [resolved]`);
145145

146146
var cmd = [...CMD_install];
147147
cmd.push("--cpanfile", cpanfile_full_path, "--installdeps", ".");
@@ -152,7 +152,7 @@ async function run() {
152152

153153
/* custom run with args */
154154
if ( has_run === false && w_args.length ) {
155-
console.log(`custom run with args`);
155+
core.info(`custom run with args`);
156156
var cmd = [...CMD_install];
157157
has_run = true;
158158
await do_exec(cmd);

0 commit comments

Comments
 (0)