Skip to content
This repository was archived by the owner on Feb 13, 2023. It is now read-only.

fix: #199 transform found Rally artifacts to upper case#264

Merged
primetheus merged 2 commits into
github:mainfrom
vorobievalex:feature/lower
May 7, 2021
Merged

fix: #199 transform found Rally artifacts to upper case#264
primetheus merged 2 commits into
github:mainfrom
vorobievalex:feature/lower

Conversation

@vorobievalex

@vorobievalex vorobievalex commented May 6, 2021

Copy link
Copy Markdown
Contributor

since Rally API returns 'undefined' errors for lowercase artifacts.
"Error occurred while validating Rally Artifacts: Error: Not able to parse artifact type: undefined"

Fixes #199

Proposed Changes

  • allow lowercase artifacts like us12345 in text, but transform it to uppercase for the Rally API

Readiness Checklist

  • If this change requires documentation, it has been included in this pull request

Reviewer Checklist

  • If a functional change has occurred, testing the integration has been performed
  • This PR has been categorized with a label (1 of automation, bug, documentation, enhancement, infrastructure, or performance) for the changelog

since Rally API returns 'undefined' errors for lowercase artifacts.

@primetheus primetheus left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@primetheus primetheus merged commit f97189a into github:main May 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error: Not able to parse artifact type: undefined

2 participants