Fix markdown list indentation in notes2news.pl script

Open
#2,972 1 comment 0 reactions 1 assignee View on GitHub

Maintainers usually reply within 3 days

@cvvergara is already working on this.

Since Sep 14, 2025.

Assessment

This issue has not been assessed yet.

Description

Problem

The NEWS.md file generated by the notes2news.pl script contains inconsistent markdown list indentation that violates markdownlint rule MD007.

Markdownlint Violations

Multiple lines in NEWS.md (lines 377-417) show indentation errors:

  • Expected: 2 spaces for nested bullets
  • Actual: 4 spaces for nested bullets

Example violations:

377-377: Unordered list indentation - Expected: 2; Actual: 4 (MD007)
381-381: Unordered list indentation - Expected: 2; Actual: 4 (MD007)
382-382: Unordered list indentation - Expected: 2; Actual: 4 (MD007)
Root Cause

If NEWS.md is generated by the notes2news.pl script, then the script needs to be updated to emit consistent 2-space indentation for nested markdown list items.

Solution

Modify notes2news.pl to:

  1. Use 2-space indentation for each nested level in markdown lists
  2. Ensure sub-bullets under top-level bullets are prefixed with exactly two spaces
  3. Maintain consistency across all generated markdown content
Reference

This issue was identified in PR #2970: https://github.com/pgRouting/pgrouting/pull/2970#discussion_r2347640433

Acceptance Criteria
  • notes2news.pl script updated to use 2-space markdown list indentation
  • Generated NEWS.md passes markdownlint MD007 rule
  • All nested bullets consistently use 2-space increments
Dominant language
C++
Stars
1.4k
Forks
397
Avg merge
2d 16h
Merged PRs (30d)
8

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from pgRouting/pgrouting

All issues in pgRouting/pgrouting

Similar issues

More C++ issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.