feat(commenting): disable if set to false

README.md

@@ -164,7 +164,7 @@ The success comment condition is generated with [Lodash template](https://lodash
 
 ##### successCommentCondition example
 
-- do no create any comments at all: `"<% return false; %>"`
+- do no create any comments at all: set to `false` or templating: `"<% return false; %>"`
 - to only comment on issues: `"<% return issue %>"`
 - to only comment on merge requests: `"<% return mergeRequest %>"`
 - you can use labels to filter issues: `"<% return issue.labels?.includes('semantic-release-relevant') %>"`
@@ -204,7 +204,7 @@ The fail comment condition is generated with [Lodash template](https://lodash.co
 
 ##### failCommentCondition example
 
-- do no create any comments at all: `"<% return false; %>"`
+- do no create any comments at all: set to `false` or templating: `"<% return false; %>"`
 - to only comment on main branch: `"<% return branch.name === 'main' %>"`
 - you can use labels to filter issues, i.e. to not comment if the issue is labeled with `wip`: `"<% return !issue.labels?.includes('wip') %>"`
 

lib/fail.js

@@ -24,6 +24,8 @@ export default async (pluginConfig, context) => {
     logger.log("Skip issue creation.");
     logger.error(`Failure reporting should be disabled via 'failCommentCondition'.
 Using 'false' for 'failComment' or 'failTitle' is deprecated and will be removed in a future major version.`);
+  } else if (failCommentCondition === false) {
+    logger.log("Skip issue creation.");
   } else {
     const encodedFailTitle = encodeURIComponent(failTitle);
     const description = failComment ? template(failComment)({ branch, errors }) : getFailComment(branch, errors);

lib/success.js

@@ -27,6 +27,8 @@ export default async (pluginConfig, context) => {
     logger.log("Skip commenting on issues and pull requests.");
     logger.error(`Issue and pull request comments should be disabled via 'successCommentCondition'.
 Using 'false' for 'successComment' is deprecated and will be removed in a future major version.`);
+  } else if (successCommentCondition === false) {
+    logger.log("Skip commenting on issues and pull requests.");
   } else {
     const releaseInfos = releases.filter((release) => Boolean(release.name));
     try {

test/fail.test.js

@@ -346,3 +346,18 @@ test.serial("Post new issue if none exists yet with disabled comment on existing
     "https://gitlab.com/test_user/test_repo/-/issues/3",
   ]);
 });
+
+test.serial("Does not post comments when failCommentCondition is set to false", async (t) => {
+  const owner = "test_user";
+  const repo = "test_repo";
+  const env = { GITLAB_TOKEN: "gitlab_token" };
+  const pluginConfig = { failCommentCondition: false };
+  const branch = { name: "main" };
+  const options = { repositoryUrl: `https://gitlab.com/${owner}/${repo}.git` };
+  const errors = [{ message: "An error occured" }];
+  const gitlab = authenticate(env);
+
+  await fail(pluginConfig, { env, options, branch, errors, logger: t.context.logger });
+
+  t.true(gitlab.isDone());
+});

test/success.test.js

@@ -289,3 +289,19 @@ test.serial(
     t.true(gitlab.isDone());
   }
 );
+
+test.serial("Does not post comments when successCommentCondition is set to false", async (t) => {
+  const owner = "test_user";
+  const repo = "test_repo";
+  const env = { GITLAB_TOKEN: "gitlab_token" };
+  const pluginConfig = { successCommentCondition: false };
+  const nextRelease = { version: "1.0.0" };
+  const releases = [{ name: RELEASE_NAME, url: "https://gitlab.com/test_user/test_repo/-/releases/v1.0.0" }];
+  const options = { repositoryUrl: `https://gitlab.com/${owner}/${repo}.git` };
+  const commits = [{ hash: "abcdef" }, { hash: "fedcba" }];
+  const gitlab = authenticate(env);
+
+  await success(pluginConfig, { env, options, nextRelease, logger: t.context.logger, commits, releases });
+
+  t.true(gitlab.isDone());
+});