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

Fix typos in PropertyAccessMode #27248

Merged
merged 1 commit into from
Jan 25, 2022
Merged

Fix typos in PropertyAccessMode #27248

merged 1 commit into from
Jan 25, 2022

Conversation

roji
Copy link
Member

@roji roji commented Jan 22, 2022

Fixes #26923

@roji roji requested a review from a team January 22, 2022 00:11
@ghost
Copy link

ghost commented Jan 22, 2022

Hello @roji!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@roji
Copy link
Member Author

roji commented Jan 24, 2022

@smitpatel any reason not to merge this? (asking since you removed auto-merge)

@smitpatel
Copy link
Contributor

/azp run

@smitpatel smitpatel closed this Jan 24, 2022
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@smitpatel smitpatel reopened this Jan 24, 2022
@smitpatel
Copy link
Contributor

@smitpatel any reason not to merge this? (asking since you removed auto-merge)

Yes, because main branch had faulty azure pipeline config, this PR didn't even run CI check other than cosmos and LCA check. #27210 was merged without actual CI check.

@ajcvickers
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@roji roji merged commit c604749 into dotnet:main Jan 25, 2022
@roji roji deleted the Typos branch January 25, 2022 15:55
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.

Repeated typo in PropertyAccessMode Enum documentation
3 participants