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

Fix Avalonia/references generation #16427

Merged
merged 2 commits into from
Jul 25, 2024

Conversation

kekekeks
Copy link
Member

@kekekeks kekekeks commented Jul 23, 2024

What does the pull request do?

Fixes intellisense in the latest version of the VS extension.

AvaloniaUI/AvaloniaVS#413 made a change to the VS extension that required the references file but #14700 removed the generation of that file (I assume accidentally). #14397 attempted to fix that but it looks like it didn't actually fix it - it was always skipped when it shouldn't be and not skipped when it should be.

This wasn't detected because 11.1.0 was tested with the then-current version of the extension which didn't include AvaloniaUI/AvaloniaVS#413. After 11.1.0 was released a new version of the extension was released, and 💥 .

@avaloniaui-bot
Copy link

You can test this PR using the following package version. 11.2.999-cibuild0050395-alpha. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

@grokys grokys added bug customer-priority Issue reported by a customer with a support agreement. labels Jul 24, 2024
@TomEdwardsEnscape
Copy link
Contributor

Ugh, nasty. I removed generation of the file because it didn't (appear to) have any further purpose once the external MSBuild processes were removed. The dependency hidden in the VS extension repo was undocumented.

@grokys
Copy link
Member

grokys commented Jul 24, 2024

The dependency hidden in the VS extension repo was undocumented.

It wasn't even there until a few months before your PR, and wasn't a part of a stable release of the extension until after 11.1.0! Just an unfortunate set of circumstances really. IMO depending on this file in the VS extension is a hack and dependencies should be detected properly, but there's no time for that right now.

@grokys grokys enabled auto-merge July 25, 2024 09:42
@avaloniaui-bot
Copy link

You can test this PR using the following package version. 11.2.999-cibuild0050430-alpha. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

@grokys grokys added this pull request to the merge queue Jul 25, 2024
Merged via the queue into master with commit 632fb91 Jul 25, 2024
11 checks passed
@grokys grokys deleted the fixes/generate-avalonia-references-with-xaml branch July 25, 2024 12:43
grokys added a commit that referenced this pull request Jul 25, 2024
Co-authored-by: Steven Kirk <grokys@users.noreply.github.com>
@Bip901

This comment was marked as off-topic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backported-11.1.x bug customer-priority Issue reported by a customer with a support agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants