Skip to content

Fix invalid YAML formatting in Dependabot workflow template#469

Merged
Marenz merged 1 commit intofrequenz-floss:v0.x.xfrom
Marenz:auto-dependabot-merge
Dec 4, 2025
Merged

Fix invalid YAML formatting in Dependabot workflow template#469
Marenz merged 1 commit intofrequenz-floss:v0.x.xfrom
Marenz:auto-dependabot-merge

Conversation

@Marenz
Copy link
Contributor

@Marenz Marenz commented Nov 19, 2025

Summary

Fixes invalid YAML syntax in the migration script's Dependabot workflow template by removing broken line continuation and unnecessary quotes on scalar values.

Copilot AI review requested due to automatic review settings November 19, 2025 13:38
@Marenz Marenz requested a review from a team as a code owner November 19, 2025 13:38
@github-actions github-actions bot added the part:cookiecutter Affects the generation of projects using cookiecutter label Nov 19, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes invalid YAML syntax in the Dependabot workflow template within the migration script by removing a broken multi-line folded scalar (>-) and replacing it with a single-line format.

Key changes:

  • Converts multi-line uses: field with folded scalar syntax to a single-line format with inline version comment
  • Removes quotes from workflow parameter values (all, true, merge, tool:auto-merged)

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

@Marenz Marenz force-pushed the auto-dependabot-merge branch 3 times, most recently from f3d6012 to 1656e86 Compare November 19, 2025 14:18
@Marenz Marenz force-pushed the auto-dependabot-merge branch 2 times, most recently from af33388 to 674a425 Compare December 2, 2025 13:20
@github-actions github-actions bot added the part:docs Affects the documentation label Dec 2, 2025
Signed-off-by: Mathias L. Baumann <mathias.baumann@frequenz.com>
@Marenz Marenz force-pushed the auto-dependabot-merge branch from 674a425 to 97e199b Compare December 2, 2025 13:25
@Marenz
Copy link
Contributor Author

Marenz commented Dec 2, 2025

It was an imperfect solution to a ci warning while trying to add the version number. This is now better.

@Marenz Marenz requested a review from llucax December 2, 2025 13:32
@Marenz Marenz added this pull request to the merge queue Dec 4, 2025
Merged via the queue into frequenz-floss:v0.x.x with commit b7b84f4 Dec 4, 2025
6 checks passed
@Marenz Marenz deleted the auto-dependabot-merge branch December 4, 2025 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

part:cookiecutter Affects the generation of projects using cookiecutter part:docs Affects the documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants