Skip to content
This repository has been archived by the owner on Feb 4, 2025. It is now read-only.

Remove parsing of error key's condition field #256

Merged
merged 2 commits into from
Jun 14, 2021

Conversation

epapbak
Copy link
Collaborator

@epapbak epapbak commented Jun 14, 2021

Description

Condition field has been deprecated from the provided rules. This commit removes the related code in this service.

Fixes #255

Type of change

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

Testing steps

Running UTs + testing the service manually

Checklist

  • make before_commit passes
  • updated documentation wherever necessary
  • added or modified tests if necessary
  • updated schemas and validators in insights-data-schemas in case of input/output change

Copy link
Collaborator

@tisnik tisnik left a comment

Choose a reason for hiding this comment

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

Is not docs generated automatically on CI?

@epapbak
Copy link
Collaborator Author

epapbak commented Jun 14, 2021

Is not docs generated automatically on CI?

It is, it is =)

Copy link
Collaborator

@tisnik tisnik left a comment

Choose a reason for hiding this comment

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

LGTM

@epapbak epapbak force-pushed the remove_metadata_condition branch from 7233357 to a55e695 Compare June 14, 2021 08:53
@epapbak epapbak force-pushed the remove_metadata_condition branch from a55e695 to 2dbc776 Compare June 14, 2021 09:37
Copy link
Collaborator

@Bee-lee Bee-lee left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

@epapbak epapbak merged commit 366b353 into RedHatInsights:master Jun 14, 2021
@epapbak epapbak deleted the remove_metadata_condition branch June 14, 2021 10:05
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.

Stop parsing condition field
3 participants