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

Update DataViewRowCursor.md #6855

Merged
merged 2 commits into from
Oct 19, 2023
Merged

Update DataViewRowCursor.md #6855

merged 2 commits into from
Oct 19, 2023

Conversation

Akash190104
Copy link
Contributor

fixed some errors and typos.

We are excited to review your PR.

So we can do the best job, please check:

  • There's a descriptive title that will make sense to other developers some time from now.
  • There's associated issues. All PR's should have issue(s) associated - unless a trivial self-evident change such as fixing a typo. You can use the format Fixes #nnnn in your description to cause GitHub to automatically close the issue(s) when your PR is merged.
  • Your change description explains what the change does, why you chose your approach, and anything else that reviewers should know.
  • You have included any necessary tests in the same PR.

fixed some errors and typos.
@codecov
Copy link

codecov bot commented Oct 7, 2023

Codecov Report

Merging #6855 (c9e7b68) into main (64d7ebd) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #6855   +/-   ##
=======================================
  Coverage   69.63%   69.63%           
=======================================
  Files        1237     1237           
  Lines      247617   247617           
  Branches    25436    25436           
=======================================
+ Hits       172429   172433    +4     
+ Misses      68577    68570    -7     
- Partials     6611     6614    +3     
Flag Coverage Δ
Debug 69.63% <ø> (+<0.01%) ⬆️
production 64.18% <ø> (+<0.01%) ⬆️
test 88.89% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 7 files with indirect coverage changes

@ericstj ericstj merged commit a3d3813 into dotnet:main Oct 19, 2023
1 of 2 checks passed
@ericstj
Copy link
Member

ericstj commented Oct 19, 2023

Thank you for your contributon!

@ghost ghost locked as resolved and limited conversation to collaborators Nov 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants