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

Schema "Mixins" do not currently support child folders #1151

Closed
jackofallops opened this issue Jul 18, 2022 · 1 comment
Closed

Schema "Mixins" do not currently support child folders #1151

jackofallops opened this issue Jul 18, 2022 · 1 comment
Labels
tool/importer-rest-api-specs Swagger Data Importer issues

Comments

@jackofallops
Copy link
Member

ctx - https://github.com/hashicorp/pandora/pull/1138/files#diff-4caec5430a3f04e5cd0cbed629b9076eda140b06cdacc11654cd0f9679a460c5R87

func findAndMergeLocalMixins(input *loads.Document, basePath string, baseFile string) (*loads.Document, error) {
	if len(strings.Split(baseFile, "/")) != 2 { // We only care about local files, not sub-folders
		return input, nil
	}

The mixins used for capturing models in referenced files currently do not support files outside the CWD. This may or may not be a problem for services such as DataFactory, this issue is to surface that this may be a problem when that service (or other service that makes use of sub-directories for additional hierarchical data) when we include them.

@tombuildsstuff tombuildsstuff added the tool/importer-rest-api-specs Swagger Data Importer issues label Jun 8, 2023
@tombuildsstuff
Copy link
Contributor

Consolidating this into #2551 since this is related to Data Factory

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tool/importer-rest-api-specs Swagger Data Importer issues
Projects
None yet
Development

No branches or pull requests

2 participants