fix(schema): Use sample instead of find for schema sampling #2481
kmruizcheck.yml
on: pull_request
check-style
2m 40s
check-generate
1m 29s
Check dependencies
1m 51s
Annotations
10 errors and 13 warnings
|
|
|
|
|
|
|
|
|
|
|
check-style:
tests/integration/tools/mongodb/metadata/collectionSchema.test.ts#L18
Replace `⏎········integration,⏎········"collection-schema",⏎········"Describe·the·schema·for·a·collection",⏎·······` with `integration,·"collection-schema",·"Describe·the·schema·for·a·collection",`
|
|
check-style:
src/tools/mongodb/metadata/collectionSchema.ts#L51
Replace `⏎················`${header}${cappedWarning}`,⏎················JSON.stringify(schema)⏎············` with ``${header}${cappedWarning}`,·JSON.stringify(schema)`
|
|
check-style:
src/tools/mongodb/metadata/collectionSchema.ts#L48
Replace `·cappedBy·!==·undefined·?·`\nThe·schema·was·inferred·from·a·subset·of·documents·due·to·the·response·size·limit.·(${cappedBy})`` with `⏎············cappedBy·!==·undefined⏎················?·`\nThe·schema·was·inferred·from·a·subset·of·documents·due·to·the·response·size·limit.·(${cappedBy})`⏎···············`
|
|
check-style:
src/tools/mongodb/metadata/collectionSchema.ts#L26
Replace `{·$sample:·{·size:·Math.min(sampleSize,·this.config.maxDocumentsPerQuery)·}·}` with `⏎············{·$sample:·{·size:·Math.min(sampleSize,·this.config.maxDocumentsPerQuery)·}·},⏎········`
|
|
check-style:
src/tools/mongodb/metadata/collectionSchema.ts#L16
Replace `.number().optional().default(ONE_MB).describe(`The·maximum·number·of·bytes·to·return·in·the·response.·This·value·is·capped·by·the·server’s·configured·maxBytesPerQuery·and·cannot·be·exceeded.`` with `⏎············.number()⏎············.optional()⏎············.default(ONE_MB)⏎············.describe(⏎················`The·maximum·number·of·bytes·to·return·in·the·response.·This·value·is·capped·by·the·server’s·configured·maxBytesPerQuery·and·cannot·be·exceeded.`⏎············`
|
|
check-generate
1 added, 0 removed; done.
|
|
check-generate
rehash: warning: skipping ca-certificates.crt,it does not contain exactly one certificate or CRL
|
|
check-generate
Updating certificates in /etc/ssl/certs...
|
|
check-generate
WARNING: apt does not have a stable CLI interface. Use with caution in scripts.
|
|
Check dependencies
1 added, 0 removed; done.
|
|
Check dependencies
rehash: warning: skipping ca-certificates.crt,it does not contain exactly one certificate or CRL
|
|
Check dependencies
Updating certificates in /etc/ssl/certs...
|
|
Check dependencies
WARNING: apt does not have a stable CLI interface. Use with caution in scripts.
|
|
check-style
1 added, 0 removed; done.
|
|
check-style
rehash: warning: skipping ca-certificates.crt,it does not contain exactly one certificate or CRL
|
|
check-style
Updating certificates in /etc/ssl/certs...
|
|
check-style
WARNING: Retrying (Retry(total=4, connect=None, read=None, redirect=None, status=None)) after connection broken by 'ProtocolError('Connection aborted.', ConnectionResetError(104, 'Connection reset by peer'))': /simple/mitmproxy/
|
|
check-style
WARNING: apt does not have a stable CLI interface. Use with caution in scripts.
|
Artifacts
Produced during runtime
| Name | Size | Digest | |
|---|---|---|---|
|
check-dep-permissions-b1a620e82fc6b882e439aaba490da5ed
|
140 Bytes |
sha256:7b3e49034b727b7437ec2edbfb7ff04bdc30b432e4a14c0ec5b26105f960146c
|
|
|
check-generate-permissions-f8999c1cd8fa5fb3d54219c9bec09212
|
140 Bytes |
sha256:13347357fff8fae5feb8cf594cd5955fd880b421939d479872432b70e4d66f99
|
|
|
check-style-permissions-dd7245b17e11397e6b3ac5ff49dd831c
|
140 Bytes |
sha256:48362ca322029a9f3b77f65b0d8ebc93461ae11863e6ab49dd98f2e1dbf7bb0e
|
|