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

#1978 - Add a few XML docs and correct class names #2556

Merged
merged 2 commits into from
Sep 10, 2021
Merged

#1978 - Add a few XML docs and correct class names #2556

merged 2 commits into from
Sep 10, 2021

Conversation

Christian-Oleson
Copy link
Contributor

@Christian-Oleson Christian-Oleson commented Sep 10, 2021

Description of Change

Issue #1978 has the request to add XML docs.

Bugs Fixed

None

API Changes

None

Added:
None

Changed:
None

Behavioral Changes

None

PR Checklist

  • Has tests (if omitted, state reason in description)
  • Rebased on top of master at time of PR
  • Changes adhere to coding standard

@dnfadmin
Copy link

dnfadmin commented Sep 10, 2021

CLA assistant check
All CLA requirements met.

src/Forms/Prism.Forms/Ioc/ContainerRegistryExtensions.cs Outdated Show resolved Hide resolved
src/Forms/Prism.Forms/Ioc/ContainerRegistryExtensions.cs Outdated Show resolved Hide resolved
src/Forms/Prism.Forms/Navigation/IInitialize.cs Outdated Show resolved Hide resolved
src/Forms/Prism.Forms/Navigation/IInitializeAsync.cs Outdated Show resolved Hide resolved
src/Forms/Prism.Forms/Navigation/IInitialize.cs Outdated Show resolved Hide resolved
src/Forms/Prism.Forms/Navigation/IInitializeAsync.cs Outdated Show resolved Hide resolved
2 More Concerns


#1978 - 2 More Concerns
@dansiegel dansiegel merged commit 883f964 into PrismLibrary:master Sep 10, 2021
@dansiegel
Copy link
Member

Thanks for the PR

@Christian-Oleson Christian-Oleson deleted the enhancement/#1978_Add_XML_Docs_And_Minor_Corrections branch September 20, 2021 09:48
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.

4 participants