Skip to content

Update ICustomMarshaler docs to talk a little bit about interaction with SetLastError#6880

Merged
elinor-fung merged 4 commits into
mainfrom
jkoritzinsky/icustommarshaler-last-pinvoke-error
Jul 16, 2021
Merged

Update ICustomMarshaler docs to talk a little bit about interaction with SetLastError#6880
elinor-fung merged 4 commits into
mainfrom
jkoritzinsky/icustommarshaler-last-pinvoke-error

Conversation

@jkoritzinsky

@jkoritzinsky jkoritzinsky commented Jul 12, 2021

Copy link
Copy Markdown
Member

@jkoritzinsky jkoritzinsky changed the title Update ICustomMarshaler docs to talk a little bit about interaction w… Update ICustomMarshaler docs to talk a little bit about interaction with SetLastError Jul 12, 2021
Comment thread xml/System.Runtime.InteropServices/ICustomMarshaler.xml Outdated
Comment thread xml/System.Runtime.InteropServices/ICustomMarshaler.xml Outdated
Co-authored-by: Elinor Fung <elfung@microsoft.com>
Comment thread xml/System.Runtime.InteropServices/ICustomMarshaler.xml Outdated
@opbld34

opbld34 commented Jul 12, 2021

Copy link
Copy Markdown

Docs Build status updates of commit 2c9d6ac:

⚠️ Validation status: warnings

File Status Preview URL Details
xml/System.Runtime.InteropServices/ICustomMarshaler.xml ⚠️Warning View Details

xml/System.Runtime.InteropServices/ICustomMarshaler.xml

  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'System.Runtime.InteropServices.Marshal.GetLastPInvokeError'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'System.Runtime.InteropServices.Marshal.GetLastPInvokeError'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'System.Runtime.InteropServices.Marshal.SetLastPInvokeError'.

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:

@jkoritzinsky

Copy link
Copy Markdown
Member Author

It looks like the docs don't have entries for Get/SetLastPInvokeError for some reason...

@elinor-fung

Copy link
Copy Markdown
Member

Looks like the docs are currently .NET 6.0 Preview 3 (which doesn't have Get/SetLastPInvokeError).

@gewarren would you be the one that knows what the expectation and process is for what preview version the docs target?

@gewarren

Copy link
Copy Markdown
Collaborator

@safern provides me with the reference assemblies for the preview releases. We were hoping to update for Preview 4 but he was out on leave. Santi, it would be good to update these again for Preview 6.

@opbld30

opbld30 commented Jul 12, 2021

Copy link
Copy Markdown

Docs Build status updates of commit a2ff349:

⚠️ Validation status: warnings

File Status Preview URL Details
xml/System.Runtime.InteropServices/ICustomMarshaler.xml ⚠️Warning View Details

xml/System.Runtime.InteropServices/ICustomMarshaler.xml

  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'System.Runtime.InteropServices.Marshal.GetLastPInvokeError'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'System.Runtime.InteropServices.Marshal.GetLastPInvokeError'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'System.Runtime.InteropServices.Marshal.SetLastPInvokeError'.

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:

@jkoritzinsky

Copy link
Copy Markdown
Member Author

After #6890 is merged, this should be updated from main. Then the validation should come back green.

@gewarren gewarren closed this Jul 16, 2021
@gewarren gewarren reopened this Jul 16, 2021
@opbld31

opbld31 commented Jul 16, 2021

Copy link
Copy Markdown

Docs Build status updates of commit a2ff349:

⚠️ Validation status: warnings

File Status Preview URL Details
xml/System.Runtime.InteropServices/ICustomMarshaler.xml ⚠️Warning View Details

xml/System.Runtime.InteropServices/ICustomMarshaler.xml

  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'System.Runtime.InteropServices.Marshal.GetLastPInvokeError'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'System.Runtime.InteropServices.Marshal.GetLastPInvokeError'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'System.Runtime.InteropServices.Marshal.SetLastPInvokeError'.

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:

Comment thread xml/System.Runtime.InteropServices/ICustomMarshaler.xml Outdated
Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
@opbld30

opbld30 commented Jul 16, 2021

Copy link
Copy Markdown

Docs Build status updates of commit acf665e:

✅ Validation status: passed

File Status Preview URL Details
xml/System.Runtime.InteropServices/ICustomMarshaler.xml ✅Succeeded View

For more details, please refer to the build report.

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.

For any questions, please:

@elinor-fung elinor-fung merged commit 229d17e into main Jul 16, 2021
@elinor-fung elinor-fung deleted the jkoritzinsky/icustommarshaler-last-pinvoke-error branch July 16, 2021 17:17
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.

Litany of bugs with ICustomMarshaler

6 participants