Skip to content
This repository was archived by the owner on Jan 12, 2026. It is now read-only.

Set spectral rule result spec to after#2147

Merged
niclim merged 1 commit intomainfrom
fix/spectral-rules
Jul 21, 2023
Merged

Set spectral rule result spec to after#2147
niclim merged 1 commit intomainfrom
fix/spectral-rules

Conversation

@niclim
Copy link
Contributor

@niclim niclim commented Jul 20, 2023

🍗 Description

What does this PR do? Anything folks should know?

I think this will actually fix #2135

What's happening is that:

  • we run spectral rules and we specify that the rule failure is from the before spec - which is incorrect as it should be from the after spec
  • when we try to group this rule into the relevant node, it cannot be associated with the right tree and fails to find where it should go
  • because of this, it doesn't render

📚 References

Links to relevant docs (Notion, Twist, GH issues, etc.), if applicable.

👹 QA

How can other humans verify that this PR is correct?

@niclim niclim requested a review from acunniffe July 20, 2023 17:12
@niclim niclim added this pull request to the merge queue Jul 21, 2023
Merged via the queue into main with commit dfc79c6 Jul 21, 2023
@niclim niclim deleted the fix/spectral-rules branch July 21, 2023 13:56
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.

Output of optic diff looks like a successful run, but it hides the errors that causes exit code 1

2 participants