Skip to content

Conversation

@wenz
Copy link
Contributor

@wenz wenz commented Oct 27, 2022

Summary

Remove incorrect Decrypt() documentation

Fixes #8322

@wenz wenz requested a review from a team as a code owner October 27, 2022 08:12
@ghost ghost added community-contribution Indicates that the PR has been added by a community member area-System.IO labels Oct 27, 2022
@ghost
Copy link

ghost commented Oct 27, 2022

Tagging subscribers to this area: @dotnet/area-system-io
See info in area-owners.md if you want to be subscribed.

Issue Details

Summary

Remove incorrect Decrypt() documentation

Fixes #8322

Author: wenz
Assignees: -
Labels:

area-System.IO, community-contribution

Milestone: -

@opbld34
Copy link

opbld34 commented Oct 27, 2022

Learn Build status updates of commit b24a5c6:

✅ Validation status: passed

File Status Preview URL Details
xml/System.IO/File.xml ✅Succeeded View
xml/System.Xml/XmlWriter.xml ✅Succeeded View
xml/System.Xml/XmlWriterSettings.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:

Copy link
Member

@adamsitnik adamsitnik left a comment

Choose a reason for hiding this comment

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

Hi @wenz

Thank you for your PR!

Before I review this PR I need to clarify the expected behavior for the decrypt method (#8322 (comment)).

Once I do that, I am going to review the PR. Thank you for your contribution!

@gewarren
Copy link
Contributor

Can you remove the System.Xml files from this PR? Those are for #8587 I believe.

@wenz
Copy link
Contributor Author

wenz commented Oct 28, 2022

Can you remove the System.Xml files from this PR? Those are for #8587 I believe.

Ugh, yes indeed. Thanks, and sorry for that! Should now be fixed.

@opbld32
Copy link

opbld32 commented Oct 28, 2022

Learn Build status updates of commit 6424ab5:

✅ Validation status: passed

File Status Preview URL Details
xml/System.IO/File.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:

@gewarren gewarren closed this Jun 5, 2024
@gewarren gewarren reopened this Jun 5, 2024
@gewarren gewarren enabled auto-merge (squash) June 5, 2024 21:29
@learn-build-service-prod
Copy link

Learn Build status updates of commit 6424ab5:

✅ Validation status: passed

File Status Preview URL Details
xml/System.IO/File.xml ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@gewarren gewarren merged commit 7c60df2 into dotnet:main Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-System.IO community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Documentation for File.Decrypt(String) Method is incorrect

5 participants