Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Validate spec only when connection is established #1223

Merged
merged 2 commits into from
Sep 20, 2023

Conversation

candiduslynx
Copy link
Contributor

@candiduslynx candiduslynx commented Sep 20, 2023

Documentation generation will fail otherwise.
Follow-up for #1214

@github-actions
Copy link

github-actions bot commented Sep 20, 2023

⏱️ Benchmark results

  • Glob-8 ns/op: 97.14

plugin/plugin.go Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Sep 20, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (2725ed5) 48.63% compared to head (00de940) 48.63%.

❗ Current head 00de940 differs from pull request most recent head da71fd4. Consider uploading reports for the commit da71fd4 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1223   +/-   ##
=======================================
  Coverage   48.63%   48.63%           
=======================================
  Files          88       88           
  Lines        8262     8262           
=======================================
  Hits         4018     4018           
  Misses       3869     3869           
  Partials      375      375           
Files Changed Coverage Δ
plugin/plugin.go 28.75% <0.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@candiduslynx candiduslynx changed the title fix: Validate only not empty inputs fix: Validate spec only when connection is established Sep 20, 2023
@github-actions github-actions bot added fix and removed fix labels Sep 20, 2023
Copy link
Member

@hermanschaaf hermanschaaf left a comment

Choose a reason for hiding this comment

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

We could probably do with a better name for NoConnection but this works

@kodiakhq kodiakhq bot merged commit 59aef16 into main Sep 20, 2023
8 checks passed
@kodiakhq kodiakhq bot deleted the fix/json-schema/empty branch September 20, 2023 10:49
kodiakhq bot pushed a commit that referenced this pull request Sep 20, 2023
🤖 I have created a release *beep* *boop*
---


## [4.9.1](v4.9.0...v4.9.1) (2023-09-20)


### Bug Fixes

* Validate spec only when connection is established ([#1223](#1223)) ([59aef16](59aef16))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants