Skip to content

Conversation

jpinz
Copy link
Contributor

@jpinz jpinz commented Aug 4, 2025

Improve error handling in docker yaml file parsing by logging the error message and passing the message to DependencyFileNotParseable

What are you trying to accomplish?

Improve debugging experience for when a yaml file fails to parse in the docker ecosystem.

How will you know you've accomplished your goal?

Errors when parsing yaml files in the docker ecosystem are logged in the output, and sent with the error as the message.

Checklist

  • I have run the complete test suite to ensure all tests and linters pass.
  • I have thoroughly tested my code changes to ensure they work as expected, including adding additional tests for new functionality.
  • I have written clear and descriptive commit messages.
  • I have provided a detailed description of the changes in the pull request, including the problem it addresses, how it fixes the problem, and any relevant details about the implementation.
  • I have ensured that the code is well-documented and easy to understand.

…or message and passing the message to DependencyFileNotParseable
@jpinz jpinz self-assigned this Aug 4, 2025
@jpinz jpinz requested a review from a team as a code owner August 4, 2025 20:21
@github-actions github-actions bot added the L: docker Docker containers label Aug 4, 2025
brettfo
brettfo previously approved these changes Aug 4, 2025
@jpinz jpinz enabled auto-merge August 5, 2025 16:21
@JamieMagee JamieMagee disabled auto-merge August 5, 2025 16:21
@jpinz jpinz merged commit 75dcfa2 into dependabot:main Aug 5, 2025
48 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L: docker Docker containers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants