Skip to content

Conversation

@DiskCrasher
Copy link
Contributor

Renamed dataAdpater variable to dataAdapter.

Summary

Fixed misspellings.

Renamed dataAdpater variable to dataAdapter.
@dotnet-bot dotnet-bot added this to the December 2021 milestone Dec 7, 2021
@BillWagner BillWagner enabled auto-merge (squash) December 7, 2021 15:15
@BillWagner BillWagner disabled auto-merge December 7, 2021 15:24
@BillWagner
Copy link
Member

Thanks for making this change @DiskCrasher

This is one of our older samples that was migrated from an internal system. Now that everything is in GitHub, we've added a CI system for our samples. That CI build is failing because this code doesn't have a .csproj file associated with. If you add a default project file in the same folder, that should fix the error.

Let me know if you can do that. If you don't have time, I'll add it and then merge this PR.

Thanks again for improving our samples.

@DiskCrasher
Copy link
Contributor Author

Please do.

@BillWagner
Copy link
Member

@DiskCrasher I don't have write access to your fork. So, I made this PR: https://github.com/DiskCrasher/docs/pull/1

If you merge that, this PR will update and I'll merge it.

@dnfadmin
Copy link

dnfadmin commented Dec 9, 2021

CLA assistant check
All CLA requirements met.

Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

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

Thanks again @DiskCrasher

The build is clean, and I'll :shipit: now.

@BillWagner BillWagner merged commit 1ddb0bd into dotnet:main Dec 9, 2021
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