Closed josephmyers closed 3 weeks ago
Palaso.sln
line 30 at r1 (raw file):
Now I'm wondering if we should actually be trying to split this stuff into Forms vs. non-Forms DLLs. None of the unit tests depend on WinForms, so all the classes they are testing could be in a non-Forms dependent DLL. I'm not sure if they do anything useful enough on their own to warrant this though.
Looking at the code, the ArchivingDlg classes (abstract base and two concrete implementations) are the only things that really need to depend on WinForms. The models do, but only minimally and in a way they really shouldn't. If we were to fix those design issues, then those models could be used to back a non WinForms implementation.
@tombogle, any chance we'll see progress here this week or next?
Not sure when I'll be able to get to this.
I discovered today that SIL.Media is a WinForms project. So a bit more to come on this
I discovered today that SIL.Media is a WinForms project. So a bit more to come on this
@josephmyers we can rename SIL.Media in a separate PR.
I discovered today that SIL.Media is a WinForms project. So a bit more to come on this
@josephmyers we can rename SIL.Media in a separate PR.
I'm doing some fixing in this area currently, so this might actually be a good time for me to try to break it into two parts. Give me a couple days, and if I can't, then we'll go ahead and merge as is.
17 files ±0 17 suites ±0 9m 34s :stopwatch: - 2m 50s 4 890 tests +6 4 659 :white_check_mark: +6 231 :zzz: ±0 0 :x: ±0 4 909 runs +6 4 665 :white_check_mark: +6 244 :zzz: ±0 0 :x: ±0
Results for commit 67b1ecd1. ± Comparison against base commit d85edaa5.
:recycle: This comment has been updated with latest results.
I have completed my work on splitting this into two DLLs. Ready for review.
Palaso.sln
line 30 at r1 (raw file):
@tombogle how about we just leave this PR as a simple rename and if you'd like to circle back on a refactor that you can do that in a separate PR? I'd like to get this merged.
No longer relevant
What sort of issues?
Never mind. I see there are some github comments that Reviewable knows nothing about.
This change is![Reviewable](https://reviewable.io/review_button.svg)