Skip to content

chore: fix MD001#5690

Merged
remyleone merged 1 commit into
mainfrom
MD001
Jun 15, 2026
Merged

chore: fix MD001#5690
remyleone merged 1 commit into
mainfrom
MD001

Conversation

@remyleone

Copy link
Copy Markdown
Member

Community Note

  • Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request.
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for pull request followers and do not help prioritize the request

Relates OR Closes #0000

Release note for CHANGELOG:


Copilot AI review requested due to automatic review settings June 15, 2026 11:35
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 50.68%. Comparing base (ff49623) to head (497e9df).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5690   +/-   ##
=======================================
  Coverage   50.68%   50.68%           
=======================================
  Files         340      340           
  Lines       78083    78083           
=======================================
  Hits        39577    39577           
  Misses      37010    37010           
  Partials     1496     1496           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

This PR updates documentation heading levels to comply with Markdown lint rule MD001 and re-enables that rule in the markdown linter configuration.

Changes:

  • Adjust heading levels in docs/developer.md to eliminate MD001 violations.
  • Re-enable MD001 by removing it from the disabled list in .rumdl.toml.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 8 comments.

File Description
docs/developer.md Updates heading levels to satisfy MD001 linting.
.rumdl.toml Stops disabling MD001 so it’s enforced going forward.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/developer.md
| `-cassettes` | Update the cassettes of the run tests |

### Objectives of the test suite
## Objectives of the test suite
Comment thread docs/developer.md
### Solution

#### Interaction Recording: Golden & Cassettes files
## Interaction Recording: Golden & Cassettes files
Comment thread docs/developer.md
Be sure to check out the resource you create in each test and remember to delete them once you need them anymore.

#### Metadata
## Metadata
Comment thread docs/developer.md
This metadata can use the `render` method to provide helpful golang templating features to have commands arguments computed dynamically.

#### BeforeFunc and AfterFunc
## BeforeFunc and AfterFunc
Comment thread docs/developer.md
Those functions can access the metadata to change dynamically their behavior.

#### Logging and debug mode
## Logging and debug mode
Comment thread docs/developer.md
When you are developing tests, you can also use the `Logger` field that is available in the different contexts to write your own logs.

### Checking the version
## Checking the version
Comment thread docs/developer.md
When you are running CLI commands the version check is made in every action. You can avoid these output setting `SCW_DISABLE_CHECK_VERSION` to false.

#### Targeting specific tests
## Targeting specific tests
Comment thread docs/developer.md
Always run the test on the whole package (here the "baremetal" package stored in the folder "./internal/namespaces/baremetal/v1") and use the `-run` to target specific tests.

### Adding new tests
## Adding new tests
@remyleone remyleone added this pull request to the merge queue Jun 15, 2026
Merged via the queue into main with commit 5800b13 Jun 15, 2026
124 checks passed
@remyleone remyleone deleted the MD001 branch June 15, 2026 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants