Skip to content

Commit

Permalink
Fix issues with no summary
Browse files Browse the repository at this point in the history
  • Loading branch information
dogle-scottlogic committed Nov 18, 2024
1 parent a46f52c commit ccfb1a8
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ tags:
- stakeholder
author: fhubin
layout: default_post
summary: In this article, I would share some thoughts about how to approach something we all dread and why it is important to be transparent in an appropriately sensitive way.
---

In a client-supplier relationship, communication is of paramount importance. And good communication requires transparency. A supplier in charge of a doomed project telling their client that all is going well would not be doing a very good job. Similarly, a client whose budget has been cut telling a supplier that all is well would also not be delivering the right message.
Expand Down
6 changes: 5 additions & 1 deletion lintPosts.js
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,11 @@ const lintPosts = () => {
const postDate = new Date(postDateString);
if (postDate > new Date("2018-03-26")) {
// Note _prose.yml specifies 130 characters are needed, so if you change this please also change the instructions
if (!summary || summary.length < 130) {
if(!summary) {
console.error("The post " + path + " does not have a summary.")
fail = true;
}
else if (summary.length < 130) {
console.warn(
"Post summary length is " + summary.length + ". Recommended minimum length for the summary is 130 characters."
);
Expand Down

0 comments on commit ccfb1a8

Please sign in to comment.