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

Copy runtime.File as swag.File #60

Merged
merged 1 commit into from
Feb 1, 2022
Merged

Conversation

kzys
Copy link
Contributor

@kzys kzys commented Jan 28, 2022

The File type is defined in go-openapi/runtime and used from
go-openapi/validate. However the runtime package uses the validate
package which makes a circular dependency between them.

go-openapi/validate#143

Signed-off-by: Kazuyoshi Kato [email protected]

@youyuanwu youyuanwu requested a review from casualjim January 29, 2022 18:05
@kzys
Copy link
Contributor Author

kzys commented Jan 29, 2022

Let me rebase the branch to skip Travis CI.

The File type is defined in go-openapi/runtime and used from
go-openapi/validate. However the runtime package uses the validate
package which makes a circular dependency between them.

go-openapi/validate#143

Signed-off-by: Kazuyoshi Kato <[email protected]>
@codecov
Copy link

codecov bot commented Jan 29, 2022

Codecov Report

Merging #60 (9230c5f) into master (9afd6d4) will decrease coverage by 0.25%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #60      +/-   ##
==========================================
- Coverage   88.84%   88.59%   -0.26%     
==========================================
  Files          12       13       +1     
  Lines        1390     1394       +4     
==========================================
  Hits         1235     1235              
- Misses        130      134       +4     
  Partials       25       25              
Impacted Files Coverage Δ
file.go 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9afd6d4...9230c5f. Read the comment docs.

@youyuanwu
Copy link
Member

@casualjim I think move File from runtime to here is ok. Please merge it if this looks ok.

@casualjim casualjim merged commit ef31b6b into go-openapi:master Feb 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants