Skip to content

Clean up from previous PR#8863

Merged
jaredpar merged 1 commit intomainfrom
dev/jaredpar/fixup
Jun 23, 2023
Merged

Clean up from previous PR#8863
jaredpar merged 1 commit intomainfrom
dev/jaredpar/fixup

Conversation

@jaredpar
Copy link
Copy Markdown
Member

This addresses some feedback from #8762



This addresses some feedback from #8762
@jaredpar jaredpar requested a review from a team as a code owner June 23, 2023 18:14
@echo off
setlocal

set _args=-restore -build /p:_SelfContainedWasSpecified=true %*
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The original code here was /p:TreatWarningsAsErros=true. This is intentionally being removed because it's redundant with the defaults for $warnAsError of build.ps1 and together it makes it hard to turn this off locally.

@jaredpar jaredpar merged commit c3de764 into main Jun 23, 2023
@jaredpar jaredpar deleted the dev/jaredpar/fixup branch June 23, 2023 21:21
333fred added a commit to 333fred/razor that referenced this pull request Jun 26, 2023
* upstream/main: (212 commits)
  Bump analyzer testing
  Clean up from previous PR (dotnet#8863)
  new zip idea
  Apply code review feedback
  powershell
  Apply suggestions from code review
  Fix test failures after resolving merge conflict
  More
  Update src/Razor/src/rzls/rzls.csproj
  fix container
  fix queue names
  another workaround
  Add ImmutableArray<T>.SelectAsArray(...) extension method
  Work around unsupported base image
  Return ImmutableArray<T> from GetClassifiedSpans() and GetTagHelperSpans()
  Add ItemCollection.TryGetValue(...) method
  Add ImmutableArray<T>.DrainToImmutable() extension method
  Fix version diff
  Rename extension method to match others
  Add `global::` prefix to default cshtml usings (dotnet#8857)
  ...
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.

2 participants