Skip to content

Improve MSBuild Error Actionability in Resizetizer#28999

Merged
rmarinho merged 5 commits intonet10.0from
fix-27127
May 28, 2025
Merged

Improve MSBuild Error Actionability in Resizetizer#28999
rmarinho merged 5 commits intonet10.0from
fix-27127

Conversation

@jsuarezruiz
Copy link
Contributor

Description of Change

Improve MSBuild Error Actionability in Resizetizer. All the error used the MAUI0000 code. With the changes in this PR, the error codes have been updated to use a new format MAUIR to indicate resizetizer as the category, and have a few new code numbers for the different types of errors in that.

Code Name Severity Description Category
MAUIR0001 ResizetizeImages Error High Unknown error occurred while processing a particular image file. Image Processing . Bubble-up errors during image processing.
MAUIR0002 Specific Image File Error High Unknown error occurred while processing a particular image file. Image Processing
MAUIR0003 MauiAsyncTask Error Medium Unknown error occurred during resource processing. General Resource Processing
MAUIR0004 Android Resources Error High Error in generating/writing Android-specific resources. Android Resource Processing
MAUIR0005 iOS Storyboard Error High Issue occurred while processing iOS/MacCatalyst-specific storyboard. iOS Resource Processing

Issues Fixed

Fixes #27127

@jsuarezruiz jsuarezruiz added the area-single-project Splash Screen, Multi-Targeting, MauiFont, MauiImage, MauiAsset, Resizetizer label Apr 15, 2025
Copilot AI review requested due to automatic review settings April 15, 2025 10:49
@jsuarezruiz jsuarezruiz requested a review from a team as a code owner April 15, 2025 10:49
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

catch (Exception ex)
{
Log.LogError(null, "MAUI0000", null, null, 0, 0, 0, 0, ex.ToString());
Log.LogError("Apple Resource Processing", "MAUIR0005", null, null, 0, 0, 0, 0, $"Error occurred in processing Apple-specific storyboard resources. {ex.ToString()}");
Copy link
Member

Choose a reason for hiding this comment

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

Should we add these to a Resource files so we can translate these errors using OneLoc?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Could you provide more details? Should I add .resx files for localization? If so, how many languages should be included? Should I handle the translations myself using tools or AI, or would it be better to coordinate with someone for more accurate translations?

Copy link
Member

Choose a reason for hiding this comment

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

you just add 1 resx the others will be generated by OneLoc tool . we have a service that handles the translations.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nice, moved the error messages to a resx file and also create a class with constants for the error codes.

@PureWeen PureWeen moved this from Todo to Ready To Review in MAUI SDK Ongoing May 20, 2025
@jsuarezruiz jsuarezruiz requested a review from rmarinho May 27, 2025 12:00
@github-project-automation github-project-automation bot moved this from Ready To Review to Approved in MAUI SDK Ongoing May 27, 2025
@rmarinho rmarinho merged commit 768bedb into net10.0 May 28, 2025
116 of 130 checks passed
@rmarinho rmarinho deleted the fix-27127 branch May 28, 2025 10:37
@github-project-automation github-project-automation bot moved this from Approved to Done in MAUI SDK Ongoing May 28, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Jun 28, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-single-project Splash Screen, Multi-Targeting, MauiFont, MauiImage, MauiAsset, Resizetizer

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants