-
Notifications
You must be signed in to change notification settings - Fork 4.5k
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
Reference Source repo: Resource string files missing #17174
Comments
@karelz not sure the area-System.ComponentModel.DataAnnotations label is appropriate for this issue. Do you know who could actually help with getting the resx files into the reference source repo? |
Xamarin is now migrating to use CoreFX (sources now, binaries hopefully later). |
I looked at System.ComponentModel.DataAnnotations few days ago and it looks like we won't be able to use it for some time as it's missing quite few APIs and implementation dependencies in CoreFX compare to full .NET hence even though we'd love to use it we need to first put back code which was removed in CoreFX. I'll try to use at least CoreFX resx files instead of referencesource to resolve this issue. |
@richlander @terrajobst what is the process for updating one file in https://github.com/Microsoft/referencesource? |
@karelz it looks like this should probably be assigned to someone outside of our team to come to resolution. Will you figure out the right owner? |
@divega when we know who should do the work, I agree - right now we need to get traction on what needs to happen where, so keeping it in the feature area seems most logical (e.g. we might need your team to "approve" releasing the source code). Makes sense? |
Ping... @karelz @richlander @preetikr |
Awesome. Will pass on ... |
Clearing up milestone so that we can track this for now in our triage. |
Removing this from data-annotations area because there is currently nothing actionable by the area owners. We would be happy to do the work here, but without clear guidance from people who know about these things there is nothing we can currently do. Also happy to close if the value is not high enough to get such guidance. |
@ajcvickers please always set new area label. Issues without area labels are not tracked and won't get any attention. |
This issue was moved to microsoft/referencesource#102 |
@karelz Sorry, I though that would trigger re-analysis of the area on your side. What's the best way to deal with an issue that seems to be in the wrong area if there is no other obvious area? |
@ajcvickers the best approach is to triage it. If it was misrouted to certain area, there is high likelyhood it would land there again from "general high-level" triage like I do. Area owners are expected to find better home for the issue. If not clear, you can always fall back to me for guidance / advice if it is a bit more esoteric (like this one with ReferenceSource). |
@karelz Sounds good, thanks. |
The resource file /src/System.ComponentModel.Annotations/src/Resources/Strings.resx is not part of Microsoft/referencesource which mono is using. As a result validation results on mono have the resource names being returned instead of the actual validation message. Running the same code on Windows returns proper validation messages.
The text was updated successfully, but these errors were encountered: