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

Refactor F# compile order #5095

Merged
merged 1 commit into from
Feb 21, 2025
Merged

Conversation

Youssef1313
Copy link
Member

I think it's a better idea to avoid having dependencies between the self-registered generation and the entry point generation. We let the target run on whatever order, and we still respect the compile order by using CompileAfter for the entry point.

Copy link
Member

@Evangelink Evangelink left a comment

Choose a reason for hiding this comment

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

It's better than today but I think it'd be nice if the self-registered extensions were before so that main (usually last file of the project) can use the regular Compile

@Evangelink Evangelink merged commit 1466046 into microsoft:main Feb 21, 2025
9 checks passed
@Youssef1313 Youssef1313 deleted the msbuild-cleanup branch February 21, 2025 13:40
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