Skip to content

Conversation

@cernymi
Copy link
Contributor

@cernymi cernymi commented Nov 17, 2025

Related Issue

None

New Behavior

None

...

Contrast to Current Behavior

Yamllint runs early in the pipeline and also on every push, so issues get caught immediately.

...

Discussion: Benefits and Drawbacks

The code should use cleaner YAML formatting.
The line-length.max is set to 230 to accommodate the project’s current state.
Fell free to make improvements in .yamllint.yml

Main purpose now is to get rid of training spaces

...

Changes to the Documentation

None

...

Proposed Release Note Entry

add yamllint to project pipeline

...

Double Check

  • I have read the comments and followed the CONTRIBUTING.md.
  • I have explained my PR according to the information in the comments or in a linked issue.
  • My PR targets the devel branch.

@sc68cal sc68cal merged commit 48d2d15 into netbox-community:devel Nov 17, 2025
56 checks passed
@cernymi cernymi deleted the cernymi/yamllint branch November 17, 2025 19:40
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.

2 participants