Skip to content

Avoid exception on missing profile#735

Merged
sbrunner merged 1 commit intomasterfrom
missing-profile
Mar 12, 2025
Merged

Avoid exception on missing profile#735
sbrunner merged 1 commit intomasterfrom
missing-profile

Conversation

@sbrunner
Copy link
Copy Markdown
Member

@sbrunner sbrunner commented Mar 12, 2025

Description

Avoid exception on missing profile

Related Issue

Fix #734

Motivation and Context

Have a cleaner and more understandable output

How Has This Been Tested?

prospector --profile=dont_exists.yaml
=>

Failed to run:
Could not find profile dont_exists.yaml
Search path: /home/sbrunner/workspace/prospector:/home/sbrunner/workspace/prospector/prospector/profiles/profiles, or in module 'prospector_profile_dont_exists.yaml'

prospector --profile=utils:dont_exists
=>

Failed to run:
Could not find profile utils:dont_exists.
Search path: /home/sbrunner/workspace/prospector:/home/sbrunner/workspace/prospector/prospector/profiles/profiles, or in module 'prospector_profile_utils'

prospector --profile=utils
=>

Failed to run:
Could not find profile utils.
Search path: /home/sbrunner/workspace/prospector:/home/sbrunner/workspace/prospector/prospector/profiles/profiles, or in module 'prospector_profile_utils'

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

@sbrunner sbrunner marked this pull request as ready for review March 12, 2025 12:49
@sbrunner sbrunner merged commit f059aba into master Mar 12, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Pierre-Sassoulas Pierre-Sassoulas Pierre-Sassoulas approved these changes

@carlio carlio Awaiting requested review from carlio

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[BUG] Avoid exception on missing profile

2 participants

@sbrunner @Pierre-Sassoulas