Skip to content

Improve error messages regarding track syntax errors#1908

Merged
gareth-ellis merged 3 commits intomasterfrom
improve-errors
Jan 29, 2025
Merged

Improve error messages regarding track syntax errors#1908
gareth-ellis merged 3 commits intomasterfrom
improve-errors

Conversation

@gareth-ellis
Copy link
Copy Markdown
Member

I've encountered a few cryptic error messages whilst tweeking a track.

Issues this solves:

  • If we fail to transform the templates completely due to a syntax error, we write that the full output is listed XXXX - it isnt, we never get round to writing it, so I am catching the TemplateSyntaxError and writing another error message. It would be nice if we can say what the error is, or at least where, but we can't yet. If I find a way I will add that to the PR / a new PR
  • If you don't include "operation", you get an Attribute error, but no information where in the track. Now we include a more helpful message that should help users see where the issue may be.

@gareth-ellis gareth-ellis requested a review from a team January 29, 2025 14:44
@gareth-ellis gareth-ellis added enhancement Improves the status quo :Usability Makes Rally easier to use labels Jan 29, 2025
Copy link
Copy Markdown
Contributor

@dpifke-elastic dpifke-elastic left a comment

Choose a reason for hiding this comment

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

LGTM.

@gareth-ellis gareth-ellis merged commit c5b469d into master Jan 29, 2025
@dpifke-elastic dpifke-elastic added this to the 2.12.0 milestone Mar 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Improves the status quo :Usability Makes Rally easier to use

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants