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

Add note regarding Maestro timing #3152

Merged
merged 1 commit into from
Jul 20, 2023
Merged

Add note regarding Maestro timing #3152

merged 1 commit into from
Jul 20, 2023

Conversation

sblom
Copy link
Contributor

@sblom sblom commented Jul 20, 2023

No description provided.

@DrewScoggins DrewScoggins self-requested a review July 20, 2023 19:01
Copy link
Member

@DrewScoggins DrewScoggins left a comment

Choose a reason for hiding this comment

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

LGTM

@DrewScoggins DrewScoggins self-requested a review July 20, 2023 19:01
@@ -25,3 +25,5 @@ The .NET Core SDK contains both the .NET Core runtime and CLI tools. .NET Perfor
- [Downloads](https://github.com/dotnet/core-sdk#installers-and-binaries)

Optionally, you could use [dotnet.py](../scripts/dotnet.py) to to download the DotNet Cli locally.

**NOTE:** If you see strange build breaks, consider waiting a few hours for Maestro to flow updates to this repo. If that doesn't fix it, file an issue in dotnet/performance and consider emailing dotnetperf.
Copy link
Member

Choose a reason for hiding this comment

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

We should probably put the full dotnetperf email just for complete clarity.

@sblom sblom merged commit cbd186d into main Jul 20, 2023
5 checks passed
@LoopedBard3 LoopedBard3 deleted the maestro-note branch July 20, 2023 20: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.

3 participants