Skip to content

Update build-your-first-aspire-app.md to highlight the correct line #884

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
May 13, 2024
Merged

Update build-your-first-aspire-app.md to highlight the correct line #884

merged 1 commit into from
May 13, 2024

Conversation

Sverigevader
Copy link
Contributor

@Sverigevader Sverigevader commented May 13, 2024 •
edited by github-actions bot
Loading

Summary

Highlight the correct line in the code snippet

Internal previews

📄 File 🔗 Preview link
docs/get-started/build-your-first-aspire-app.md Quickstart: Build your first .NET Aspire app

@Sverigevader Sverigevader requested a review from IEvangelist as a code owner May 13, 2024 09:07
@dotnet-bot dotnet-bot added the getting-started Content related to getting started. label May 13, 2024
@Sverigevader Sverigevader reopened this May 13, 2024
@IEvangelist
Copy link
Member

Already fixed in pending PR #870, but let's approve and merge this one anyway. Thank you @Sverigevader

@IEvangelist IEvangelist enabled auto-merge (squash) May 13, 2024 11:23
@Sverigevader
Copy link
Contributor Author

@IEvangelist Sounds good, thanks! 👍

@IEvangelist IEvangelist merged commit 9e9caf2 into dotnet:main May 13, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@IEvangelist IEvangelist IEvangelist approved these changes

Assignees
No one assigned
Labels
getting-started Content related to getting started.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Sverigevader @IEvangelist @dotnet-bot