Skip to content

remove required from non-mandatory parameters#1592

Open
soferreira wants to merge 1 commit intoMicrosoftDocs:mainfrom
soferreira:patch-1
Open

remove required from non-mandatory parameters#1592
soferreira wants to merge 1 commit intoMicrosoftDocs:mainfrom
soferreira:patch-1

Conversation

@soferreira
Copy link
Copy Markdown

indexing_directive, pre_trigger_include and post_trigger_include are not mandatory parameters

indexing_directive, pre_trigger_include and post_trigger_include are not mandatory parameters
@opbld32
Copy link
Copy Markdown

opbld32 commented Aug 2, 2022

Docs Build status updates of commit 416fbfe:

⚠️ Validation status: warnings

File Status Preview URL Details
docs-ref-autogen/azure-cosmos/azure.cosmos.container.ContainerProxy.yml ⚠️Warning View (azure-python) Details

docs-ref-autogen/azure-cosmos/azure.cosmos.container.ContainerProxy.yml

  • Line 7, Column 3: [Warning: xref-not-found] Cross reference not found: 'builtins.object'.
  • Line 77, Column 19: [Warning: xref-not-found] Cross reference not found: 'str,str'.
  • Line 93, Column 7: [Warning: xref-not-found] Cross reference not found: 'Callable'.
  • Line 97, Column 31: [Warning: xref-not-found] Cross reference not found: 'Any'.
  • Line 119, Column 7: [Warning: xref-not-found] Cross reference not found: 'Callable'.
  • Line 155, Column 19: [Warning: xref-not-found] Cross reference not found: 'str,str'.
  • Line 171, Column 7: [Warning: xref-not-found] Cross reference not found: 'Callable'.
  • Line 195, Column 7: [Warning: xref-not-found] Cross reference not found: 'Callable'.
  • Line 199, Column 31: [Warning: xref-not-found] Cross reference not found: 'Any'.
  • Line 214, Column 7: [Warning: xref-not-found] Cross reference not found: 'azure.cosmos.ThroughputProperties'.
  • Line 227, Column 7: [Warning: xref-not-found] Cross reference not found: 'Callable'.
  • Line 231, Column 7: [Warning: xref-not-found] Cross reference not found: 'Iterable'.
  • Line 231, Column 47: [Warning: xref-not-found] Cross reference not found: 'Any'.
  • Line 264, Column 7: [Warning: xref-not-found] Cross reference not found: 'Callable'.
  • Line 268, Column 7: [Warning: xref-not-found] Cross reference not found: 'Iterable'.
  • Line 268, Column 47: [Warning: xref-not-found] Cross reference not found: 'Any'.
  • Line 287, Column 7: [Warning: xref-not-found] Cross reference not found: 'ItemPaged'.
  • Line 287, Column 24: [Warning: xref-not-found] Cross reference not found: 'Dict'.
  • Line 287, Column 48: [Warning: xref-not-found] Cross reference not found: 'Any'.
  • Line 340, Column 7: [Warning: xref-not-found] Cross reference not found: 'Callable'.
  • Line 344, Column 7: [Warning: xref-not-found] Cross reference not found: 'Iterable'.
  • Line 344, Column 47: [Warning: xref-not-found] Cross reference not found: 'Any'.
  • Line 369, Column 19: [Warning: xref-not-found] Cross reference not found: 'str,str'.
  • Line 373, Column 7: [Warning: xref-not-found] Cross reference not found: 'Callable'.
  • Line 377, Column 31: [Warning: xref-not-found] Cross reference not found: 'Any'.

This comment lists only the first 25 errors (including error/warning/suggestion) in the pull request.
For more details, please refer to the build report.

If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

Copy link
Copy Markdown

@JericHunter JericHunter left a comment

Choose a reason for hiding this comment

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

LGTM!

@gahl-levy
Copy link
Copy Markdown

@seesharprun Can we please get your help to approve this? My approval isn't enough :D

@seesharprun
Copy link
Copy Markdown

I don't know if I have permissions. Let's give it a shot

#sign-off

@seesharprun
Copy link
Copy Markdown

Okay, it looks like I need to pull in the API docs team. I'll do that shortly.

@soferreira
Copy link
Copy Markdown
Author

Hi @seesharprun any update on this PR?

@simorenoh
Copy link
Copy Markdown

@seesharprun pinging on this again to see if we can get help merging this

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.

6 participants