Skip to content

Update TestRazorCompiledItem.cs #54211

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Mar 4, 2024
Merged

Update TestRazorCompiledItem.cs #54211

merged 1 commit into from
Mar 4, 2024

Conversation

WeihanLi
Copy link
Contributor

use new Convert.ToHexStringLower

Description

use new Convert.ToHexStringLower

@WeihanLi
Update TestRazorCompiledItem.cs
7ff97b6 
use Convert.ToHexStringLower
@WeihanLi WeihanLi requested a review from a team as a code owner February 25, 2024 05:49
@ghost ghost added the area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates label Feb 25, 2024
@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Feb 25, 2024
@mkArtakMSFT mkArtakMSFT requested a review from javiercn February 26, 2024 18:42
captainsafia
captainsafia approved these changes Feb 26, 2024
View reviewed changes
Copy link
Member

@captainsafia captainsafia left a comment

Choose a reason for hiding this comment

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

LGTM -- change only impacts test code and looks like overall goodness.

TY @WeihanLi!

@dotnet-policy-service dotnet-policy-service bot added the pending-ci-rerun When assigned to a PR indicates that the CI checks should be rerun label Mar 4, 2024
@captainsafia captainsafia merged commit 958c8a7 into dotnet:main Mar 4, 2024
@danroth27 danroth27 added this to the 9.0-preview3 milestone Apr 29, 2024
@WeihanLi WeihanLi deleted the patch-1 branch May 15, 2024 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@captainsafia captainsafia captainsafia approved these changes

@javiercn javiercn Awaiting requested review from javiercn

Assignees

@javiercn javiercn

Labels
area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates community-contribution Indicates that the PR has been added by a community member pending-ci-rerun When assigned to a PR indicates that the CI checks should be rerun
Projects
None yet
Milestone
9.0-preview3
Development

Successfully merging this pull request may close these issues.
4 participants
@WeihanLi @captainsafia @danroth27 @javiercn