Started by GitHub push by sarbian35283a16f1323752ae4f6d3cd4a16dffd8578e60a4283a16f1323752ae4f6d3cd4a16dffd8578e60a4refs/remotes/origin/dev283a16f1323752ae4f6d3cd4a16dffd8578e60a4283a16f1323752ae4f6d3cd4a16dffd8578e60a4refs/remotes/origin/dev283a16f1323752ae4f6d3cd4a16dffd8578e60a4283a16f1323752ae4f6d3cd4a16dffd8578e60a4refs/remotes/origin/devhttps://github.com/sarbian/ModuleManager.githttps://ksp.sarbian.com/jenkins/user/markusappel/my-views/view/all/job/ModuleManager-Dev/35/artifacthttps://ksp.sarbian.com/jenkins/user/markusappel/my-views/view/all/job/ModuleManager-Dev/changeshttps://ksp.sarbian.com/jenkins/user/markusappel/my-views/view/all/job/ModuleManager-Dev/35/https://ksp.sarbian.com/jenkins/user/markusappel/my-views/view/all/job/ModuleManager-Dev/35/testReportModuleManager.dllModuleManager.dllModuleManager.dllfalse#3565406498ModuleManager-Dev #3535falsefalse35254SUCCESS1538909340071https://ksp.sarbian.com/jenkins/user/markusappel/my-views/view/all/job/ModuleManager-Dev/35/- ModuleManager/Progress/ProgressCounter.cs7f84a464584133e780a438e7d6ccc0a4f6448a3f1530686011000https://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong@gmail.comWhitespace
2018-07-03 23:33:31 -07007f84a464584133e780a438e7d6ccc0a4f6448a3fWhitespaceeditModuleManager/Progress/ProgressCounter.cs
- ModuleManagerTests/Progress/PatchProgressTest.csModuleManager/MMPatchLoader.csModuleManager/Progress/ProgressCounter.csModuleManager/Progress/PatchProgress.csModuleManager/Progress/IPatchProgress.cs7d2507abb3a4b3d28d65cc2709f90427a5ec525b1530686325000https://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong@gmail.comAllow warnings
Cache generation can proceed but the user will be alerted.
2018-07-03 23:38:45 -07007d2507abb3a4b3d28d65cc2709f90427a5ec525bAllow warningseditModuleManager/Progress/ProgressCounter.cseditModuleManagerTests/Progress/PatchProgressTest.cseditModuleManager/Progress/IPatchProgress.cseditModuleManager/MMPatchLoader.cseditModuleManager/Progress/PatchProgress.cs
- ModuleManager/Collections/ArrayEnumerator.csModuleManagerTests/Collections/ArrayEnumeratorTest.cs106581c492eaeee390a2c35b404a299acfcbea3e1530686392000https://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong@gmail.comMake ArrayEnumerator more versatile
Now optionally accepts start index and length
2018-07-03 23:39:52 -0700106581c492eaeee390a2c35b404a299acfcbea3eMake ArrayEnumerator more versatileeditModuleManagerTests/Collections/ArrayEnumeratorTest.cseditModuleManager/Collections/ArrayEnumerator.cs
- ModuleManager/Tags/TagListParser.csModuleManager/Tags/TagList.csModuleManagerTests/Tags/TagListParserTest.csModuleManager/Tags/Tag.csModuleManagerTests/ModuleManagerTests.csprojModuleManagerTests/Tags/TagListTest.csModuleManagerTests/Tags/TagTest.csModuleManager/ModuleManager.csprojee3e340f51ebddc453a67a673f2e3fd1222040091530687203000https://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong@gmail.comTag, TagList, TagListParser
A lot of things in MM are structured like :tag[value]trailer and this
formalizes that structure
2018-07-03 23:53:23 -0700ee3e340f51ebddc453a67a673f2e3fd122204009Tag, TagList, TagListParsereditModuleManager/ModuleManager.csprojaddModuleManager/Tags/Tag.csaddModuleManagerTests/Tags/TagListTest.csaddModuleManagerTests/Tags/TagTest.csaddModuleManager/Tags/TagList.csaddModuleManager/Tags/TagListParser.cseditModuleManagerTests/ModuleManagerTests.csprojaddModuleManagerTests/Tags/TagListParserTest.cs
- ModuleManagerTests/PatchExtractorTest.csModuleManager/Patches/EditPatch.csModuleManagerTests/Patches/PassSpecifiers/AfterPassSpecifierTest.csModuleManager/PatchExtractor.csModuleManagerTests/Patches/PassSpecifiers/InsertPassSpecifierTest.csModuleManager/Patches/IPatch.csModuleManager/Progress/IPatchProgress.csModuleManagerTests/Patches/EditPatchTest.csModuleManagerTests/Patches/PassSpecifiers/ForPassSpecifierTest.csModuleManagerTests/Patches/CopyPatchTest.csModuleManagerTests/Patches/PassSpecifiers/FinalPassSpecifierTest.csModuleManager/Patches/PassSpecifiers/FirstPassSpecifier.csModuleManager/Patches/ProtoPatch.csModuleManager/Patches/ProtoPatchBuilder.csModuleManagerTests/Patches/DeletePatchTest.csModuleManagerTests/Patches/ProtoPatchBuilderTest.csModuleManager/Patches/PassSpecifiers/LegacyPassSpecifier.csModuleManager/Patches/PassSpecifiers/BeforePassSpecifier.csModuleManager/Patches/PatchCompiler.csModuleManagerTests/Progress/PatchProgressTest.csModuleManagerTests/PatchListTest.csModuleManagerTests/Patches/PassSpecifiers/FirstPassSpecifierTest.csModuleManager/Patches/PassSpecifiers/InsertPassSpecifier.csModuleManager/Patches/PassSpecifiers/ForPassSpecifier.csModuleManager/Patches/DeletePatch.csModuleManager/Patches/PassSpecifiers/IPassSpecifier.csModuleManagerTests/Patches/PatchCompilerTest.csModuleManager/NeedsChecker.csModuleManagerTests/ModuleManagerTests.csprojModuleManagerTests/NeedsCheckerTest.csModuleManager/PatchList.csModuleManagerTests/Patches/PassSpecifiers/LegacyPassSpecifierTest.csModuleManagerTests/NodeMatcherTest.csModuleManager/Patches/PassSpecifiers/FinalPassSpecifier.csModuleManager/MMPatchLoader.csModuleManager/NodeMatcher.csModuleManagerTests/PatchApplierTest.csModuleManager/Progress/PatchProgress.csModuleManager/ModuleManager.csprojModuleManager/PatchApplier.csModuleManagerTests/Patches/PassSpecifiers/BeforePassSpecifierTest.csModuleManager/Patches/CopyPatch.csModuleManager/Patches/PassSpecifiers/AfterPassSpecifier.csf5127b1158aa586a181548f67d214c091d6498981531108997000https://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong@gmail.comRestructure patch building
Lots of changes
* Parsing/validating patch is now separate code
* Less code in the patch extractor (may even be able to go away entirely
with some simplifications)
* Pass specifier is now an explicit concept
* Needs checker is now an object and has a cleaner interface
* Some things which were errors before are now just warnings
* If there is more than one pass specifier it will take the first one
and warn
* Syntax for root patch names is much more formal now, this might break
some unusual cases that are silently accepted now
2018-07-08 21:03:17 -0700f5127b1158aa586a181548f67d214c091d649898Restructure patch buildingeditModuleManager/NodeMatcher.cseditModuleManagerTests/Progress/PatchProgressTest.cseditModuleManager/Progress/PatchProgress.cseditModuleManager/NeedsChecker.cseditModuleManager/ModuleManager.csprojeditModuleManager/PatchApplier.cseditModuleManager/Patches/IPatch.csaddModuleManager/Patches/ProtoPatchBuilder.csaddModuleManagerTests/Patches/PassSpecifiers/ForPassSpecifierTest.cseditModuleManagerTests/PatchListTest.csaddModuleManagerTests/Patches/PassSpecifiers/AfterPassSpecifierTest.csaddModuleManagerTests/Patches/PassSpecifiers/InsertPassSpecifierTest.cseditModuleManager/Patches/CopyPatch.csaddModuleManager/Patches/PassSpecifiers/LegacyPassSpecifier.cseditModuleManagerTests/Patches/EditPatchTest.csaddModuleManagerTests/Patches/PassSpecifiers/LegacyPassSpecifierTest.csaddModuleManager/Patches/PassSpecifiers/BeforePassSpecifier.csaddModuleManager/Patches/PassSpecifiers/AfterPassSpecifier.cseditModuleManagerTests/Patches/PatchCompilerTest.cseditModuleManagerTests/PatchExtractorTest.cseditModuleManagerTests/Patches/DeletePatchTest.csaddModuleManagerTests/Patches/PassSpecifiers/BeforePassSpecifierTest.csaddModuleManager/Patches/PassSpecifiers/FinalPassSpecifier.cseditModuleManager/Patches/PatchCompiler.cseditModuleManager/Progress/IPatchProgress.csaddModuleManager/Patches/ProtoPatch.csaddModuleManagerTests/Patches/ProtoPatchBuilderTest.cseditModuleManager/Patches/EditPatch.cseditModuleManagerTests/NeedsCheckerTest.csaddModuleManager/Patches/PassSpecifiers/IPassSpecifier.cseditModuleManagerTests/Patches/CopyPatchTest.csaddModuleManagerTests/Patches/PassSpecifiers/FinalPassSpecifierTest.cseditModuleManager/MMPatchLoader.csaddModuleManager/Patches/PassSpecifiers/FirstPassSpecifier.cseditModuleManagerTests/NodeMatcherTest.csaddModuleManagerTests/Patches/PassSpecifiers/FirstPassSpecifierTest.cseditModuleManagerTests/ModuleManagerTests.csprojeditModuleManagerTests/PatchApplierTest.cseditModuleManager/PatchExtractor.cseditModuleManager/PatchList.csaddModuleManager/Patches/PassSpecifiers/ForPassSpecifier.csaddModuleManager/Patches/PassSpecifiers/InsertPassSpecifier.cseditModuleManager/Patches/DeletePatch.cs
- ModuleManager/Tags/TagListParser.csModuleManager/MMPatchLoader.csModuleManagerTests/PatchExtractorTest.csModuleManager/PatchExtractor.csModuleManagerTests/Tags/TagListParserTest.cs387ed74a65e0d53f62a16c5ee0ea448b047e80ce1531111150000https://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong@gmail.comMake extra colons a warning rather than an error
Been seeing a lot of these and the correct path is determinate
2018-07-08 21:39:10 -0700387ed74a65e0d53f62a16c5ee0ea448b047e80ceMake extra colons a warning rather than an erroreditModuleManager/Tags/TagListParser.cseditModuleManagerTests/Tags/TagListParserTest.cseditModuleManagerTests/PatchExtractorTest.cseditModuleManager/MMPatchLoader.cseditModuleManager/PatchExtractor.cs
githttps://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong