cloudstack icon indicating copy to clipboard operation
cloudstack copied to clipboard

pre-commit: enable `markdownlint` rule `MD018`

Open jbampton opened this issue 1 year ago • 7 comments

https://github.com/DavidAnson/markdownlint/blob/main/doc/md018.md

Description

This PR adds another check for our Markdown linter.

MD018/no-missing-space-atx No space after hash on atx style heading.

Types of changes

  • [ ] Breaking change (fix or feature that would cause existing functionality to change)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [ ] Enhancement (improves an existing feature and functionality)
  • [X] Cleanup (Code refactoring and cleanup, that may add test cases)
  • [ ] build/CI

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • [ ] Major
  • [ ] Minor

Bug Severity

  • [ ] BLOCKER
  • [ ] Critical
  • [ ] Major
  • [ ] Minor
  • [ ] Trivial

Screenshots (if appropriate):

How Has This Been Tested?

Ran locally:

pre-commit run --all-files

How did you try to break this feature and the system with this change?

jbampton avatar May 28 '24 15:05 jbampton

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 16.06%. Comparing base (6a2c7b0) to head (c12a94d). Report is 1 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #9134      +/-   ##
============================================
- Coverage     16.90%   16.06%   -0.85%     
  Complexity    12872    12872              
============================================
  Files          5248     5642     +394     
  Lines        461567   493973   +32406     
  Branches      54170    59895    +5725     
============================================
+ Hits          78050    79351    +1301     
- Misses       374880   405837   +30957     
- Partials       8637     8785     +148     
Flag Coverage Δ
uitests 4.01% <ø> (?)
unittests 16.90% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar May 28 '24 15:05 codecov[bot]

@blueorangutan package

jbampton avatar Jul 08 '24 02:07 jbampton

@jbampton a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

blueorangutan avatar Jul 08 '24 02:07 blueorangutan

Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 10276

blueorangutan avatar Jul 08 '24 03:07 blueorangutan

@blueorangutan package

jbampton avatar Oct 14 '24 12:10 jbampton

@jbampton a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

blueorangutan avatar Oct 14 '24 12:10 blueorangutan

Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 11335

blueorangutan avatar Oct 14 '24 14:10 blueorangutan

@JoaoJandre , can we merge these linting changes (mostly by @jbampton ) during freeze? I don't think any of them have code changes. At least this one hasn't .

DaanHoogland avatar Oct 28 '24 15:10 DaanHoogland

@JoaoJandre , can we merge these linting changes (mostly by @jbampton ) during freeze? I don't think any of them have code changes. At least this one hasn't .

@DaanHoogland I don't see why we would merge them during a freeze. Although they generally do not pose any risks (at least this one doesn't), they are also not really high priority, are they? I see no problem in waiting until we release to merge these.

JoaoJandre avatar Oct 29 '24 14:10 JoaoJandre

@blueorangutan package

jbampton avatar Dec 20 '24 21:12 jbampton

@jbampton a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

blueorangutan avatar Dec 20 '24 21:12 blueorangutan

Packaging result [SF]: ✖️ el8 ✖️ el9 ✖️ debian ✖️ suse15. SL-JID 11867

blueorangutan avatar Dec 20 '24 22:12 blueorangutan

@blueorangutan package

jbampton avatar Dec 25 '24 07:12 jbampton

@jbampton a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

blueorangutan avatar Dec 25 '24 07:12 blueorangutan

Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 11903

blueorangutan avatar Dec 25 '24 09:12 blueorangutan

@blueorangutan package

DaanHoogland avatar Dec 27 '24 09:12 DaanHoogland

@DaanHoogland a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

blueorangutan avatar Dec 27 '24 09:12 blueorangutan

Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 11919

blueorangutan avatar Dec 27 '24 10:12 blueorangutan