-
Notifications
You must be signed in to change notification settings - Fork 782
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
Requires tactics Approval: Restore the ability to identify CompileBefore and CompileAfter values #17838
Conversation
Co-authored-by: Kevin Ransom (msft) <[email protected]>
✅ No release notes required |
Hey, It would be nice to not break you in the future, although I am certain we will. Thanks Kevin |
We don't need package version bump, right? |
Sigh !!!! yes we do. It's been approved by tactics. |
Thank you very much @KevinRansom!
Could you please elaborate on what exactly should the PR contain? We do indeed process things like |
@KevinRansom Do you know if this is already in .NET 9? It seems it's not included in RC2. |
No ... and I don't really know why. Except, release/dev17.12 which is our 9.0.1xx branch hasn't been updated. I will work with vlad to figure out what to do. |
@auduchinok Since, it's already approved and 9.0.1xx is still open for emergencies chet just said we can merge and flow without additional approvals. So we will see if we can get this in. Thanks for checking in, I appreciate it. |
We fixed an issue that impacted SourceLink generation for F# developers, this broke an existing rider feature, that depended on the ability to identify CompileBefore and CompileAfter values. We value our users being able to use the LTS release with their current preferred dev tooling.
This PR cherry picks the two fix PRs from main into LTS.
We failed to discover this issue locally because we no longer use the feature rider depended on. @auduchinok has modified Rider to use the updated implementation but that is only in F# 9.0 and Rider needs to support LTS and that branch does not contain these fixes. that includes these cherrypicks and has verified the fixes work with Rider.