Skip to content

Add file based program tutorial #47886

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

Merged
merged 13 commits into from
Aug 19, 2025
Merged

Conversation

@dotnetrepoman dotnetrepoman bot added this to the August 2025 milestone Aug 7, 2025
@BillWagner BillWagner marked this pull request as ready for review August 8, 2025 17:43
@BillWagner BillWagner requested a review from a team as a code owner August 8, 2025 17:43
@BillWagner BillWagner requested a review from adegeo August 8, 2025 17:43
Copy link
Contributor

@adegeo adegeo left a comment

Choose a reason for hiding this comment

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

I have a lot of feedback on this one 😸 I think it's good feedback but please feel free to push back on whatever you think doesn't make sense. Let me know when you want another review.

Copy link
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 issues. Other issues are also a high priority. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

BillWagner and others added 12 commits August 14, 2025 15:51
set the `+x` permission on the file-based program.

Fix build warnings.
Use extracted code snippets instead of inline code.
Co-authored-by: Andy (Steve) De George <67293991+adegeo@users.noreply.github.com>
@BillWagner BillWagner force-pushed the file-program-tutorial branch from 4895819 to 7077e60 Compare August 14, 2025 19:51
@BillWagner
Copy link
Member Author

@adegeo I've addressed all your comments. This is ready for another review.

Copy link
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 issues. Other issues are also a high priority. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

Copy link
Contributor

@adegeo adegeo left a comment

Choose a reason for hiding this comment

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

Awesome, approved! Two minor bug fixes to address.

Co-authored-by: Andy (Steve) De George <67293991+adegeo@users.noreply.github.com>
@BillWagner BillWagner enabled auto-merge (squash) August 19, 2025 15:59
@BillWagner BillWagner merged commit f191dd7 into dotnet:main Aug 19, 2025
8 checks passed
@BillWagner BillWagner deleted the file-program-tutorial branch August 19, 2025 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[C# 14-Tutorials]: New Feature - file-based programs
2 participants