Started by GitHub push by sarbian33eef68b8d632a9aff4b8897e823f56af56ad39ed1eef68b8d632a9aff4b8897e823f56af56ad39ed1refs/remotes/origin/deveef68b8d632a9aff4b8897e823f56af56ad39ed1eef68b8d632a9aff4b8897e823f56af56ad39ed1refs/remotes/origin/deveef68b8d632a9aff4b8897e823f56af56ad39ed1eef68b8d632a9aff4b8897e823f56af56ad39ed1refs/remotes/origin/devhttps://github.com/sarbian/ModuleManager.githttps://ksp.sarbian.com/jenkins/user/azzurite/my-views/view/all/job/ModuleManager-Dev/33/artifacthttps://ksp.sarbian.com/jenkins/user/azzurite/my-views/view/all/job/ModuleManager-Dev/changeshttps://ksp.sarbian.com/jenkins/user/azzurite/my-views/view/all/job/ModuleManager-Dev/33/https://ksp.sarbian.com/jenkins/user/azzurite/my-views/view/all/job/ModuleManager-Dev/33/testReportModuleManager.dllModuleManager.dllModuleManager.dllfalse#3373036498ModuleManager-Dev #3333falsefalse33231SUCCESS1525511701978https://ksp.sarbian.com/jenkins/user/azzurite/my-views/view/all/job/ModuleManager-Dev/33/- ModuleManager/MMPatchLoader.cs5a5b3e7c542f0a0e55c0bbc72ea2ee299a4a8e621523229691000https://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong@gmail.comMake patch stopwatch local
It's not needed outside of ProcessPatch(), which contains everything it
needs to time
2018-04-08 16:21:31 -07005a5b3e7c542f0a0e55c0bbc72ea2ee299a4a8e62Make patch stopwatch localeditModuleManager/MMPatchLoader.cs
- ModuleManager/MMPatchLoader.csbedb00c654f69cfe2ecf87b29bc25190d048901a1523229691000https://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong@gmail.comMake useCache local
Really only needed for a few lines in ProcessPatch()
2018-04-08 16:21:31 -0700bedb00c654f69cfe2ecf87b29bc25190d048901aMake useCache localeditModuleManager/MMPatchLoader.cs
- ModuleManagerTests/PatchExtractorTest.csModuleManagerTests/PatchTest.csModuleManager/PatchExtractor.csModuleManagerTests/ModuleManagerTests.csprojModuleManager/Patch.csModuleManagerTests/PatchApplierTest.csModuleManager/PatchList.csModuleManager/ModuleManager.csprojModuleManager/PatchApplier.cs5ab92f618ac9e51ebc5bb55a86ab3244ea28f47d1523229838000https://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong@gmail.comMake Patch an object
Saves having to parse things multiple times, error/info messages can now
show the original name (except NEEDS, will be worked on). More
functionality will be added to this class in future commits.
2018-04-08 16:23:58 -07005ab92f618ac9e51ebc5bb55a86ab3244ea28f47dMake Patch an objectaddModuleManager/Patch.cseditModuleManager/ModuleManager.csprojeditModuleManagerTests/ModuleManagerTests.csprojeditModuleManagerTests/PatchApplierTest.cseditModuleManager/PatchList.cseditModuleManager/PatchApplier.csaddModuleManagerTests/PatchTest.cseditModuleManager/PatchExtractor.cseditModuleManagerTests/PatchExtractorTest.cs
- ModuleManager/PatchExtractor.csa551c2cdfcf41a9494f30ba77f86f310744a78b31523229838000https://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong@gmail.comSimplify a bit
2018-04-08 16:23:58 -0700a551c2cdfcf41a9494f30ba77f86f310744a78b3Simplify a biteditModuleManager/PatchExtractor.cs
- ModuleManager/PatchExtractor.csf96c333cb23db5a6d54c91faedbb088a5791899d1523229838000https://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong@gmail.comExtra semicolon
2018-04-08 16:23:58 -0700f96c333cb23db5a6d54c91faedbb088a5791899dExtra semicoloneditModuleManager/PatchExtractor.cs
- ModuleManager/MMPatchLoader.csModuleManagerTests/PatchExtractorTest.csModuleManager/PatchExtractor.csModuleManagerTests/ModuleManagerTests.csprojModuleManagerTests/PatchListTest.csModuleManager/Collections/ArrayEnumerator.csModuleManager/Pass.csModuleManager/PatchList.csModuleManagerTests/PatchApplierTest.csModuleManager/ModuleManager.csprojModuleManager/PatchApplier.csModuleManagerTests/PassTest.cs83288b5773b3115ce6870e6107a8f0eb1d97703b1523510679000https://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong@gmail.comExtract interface for PatchList, modify PatchExtractor
PatchExtractor did too much at once, it shouldn't enumerate and extract
at the same time (still some work to be done there...). PatchList now
has a clearer interface and obscures more of the details - iterating
through patches is now handled by it rater than PatchApplier
2018-04-11 22:24:39 -070083288b5773b3115ce6870e6107a8f0eb1d97703bExtract interface for PatchList, modify PatchExtractoreditModuleManager/ModuleManager.csprojeditModuleManagerTests/PatchExtractorTest.cseditModuleManager/Collections/ArrayEnumerator.cseditModuleManager/MMPatchLoader.cseditModuleManager/PatchExtractor.cseditModuleManagerTests/ModuleManagerTests.csprojeditModuleManager/PatchApplier.csaddModuleManagerTests/PassTest.csaddModuleManager/Pass.cseditModuleManager/PatchList.cseditModuleManagerTests/PatchApplierTest.cseditModuleManagerTests/PatchListTest.cs
- ModuleManager/MMPatchLoader.cscccdc85fcb522b7be1f0682baf8807934b43cea11523510679000https://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong@gmail.comAllow null value in wildcard match
Allows for instance :HAS[#someValue] instead of :HAS[#someValue[*]]
2018-04-11 22:24:39 -0700cccdc85fcb522b7be1f0682baf8807934b43cea1Allow null value in wildcard matcheditModuleManager/MMPatchLoader.cs
- ModuleManagerTests/PatchExtractorTest.csModuleManagerTests/PatchTest.csModuleManagerTests/ModuleManagerTests.csprojModuleManager/NodeMatcher.csModuleManager/Patch.csModuleManager/PatchApplier.csModuleManager/ModuleManager.csprojModuleManagerTests/NodeMatcherTest.cs3698d7c53885fb090ed1ef7b5ba5b4fedc22a3771523512958000https://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong@gmail.comExtract node matcher
Will eventually be useful in modifying nodes too, but not without some
refactoring. For now it is only used on root patches.
2018-04-11 23:02:38 -07003698d7c53885fb090ed1ef7b5ba5b4fedc22a377Extract node matchereditModuleManager/PatchApplier.csaddModuleManagerTests/NodeMatcherTest.cseditModuleManagerTests/PatchExtractorTest.cseditModuleManagerTests/ModuleManagerTests.csprojaddModuleManager/NodeMatcher.cseditModuleManagerTests/PatchTest.cseditModuleManager/ModuleManager.csprojeditModuleManager/Patch.cs
- ModuleManager/Patches/DeletePatch.csModuleManagerTests/PatchExtractorTest.csModuleManager/Patches/EditPatch.csModuleManagerTests/Patches/PatchCompilerTest.csModuleManager/PatchExtractor.csModuleManager/Patches/IPatch.csModuleManagerTests/ModuleManagerTests.csprojModuleManagerTests/Patches/EditPatchTest.csModuleManagerTests/Patches/CopyPatchTest.csModuleManager/Patch.csModuleManager/PatchList.csModuleManager/Pass.csModuleManagerTests/Patches/DeletePatchTest.csModuleManagerTests/PatchTest.csModuleManager/Patches/PatchCompiler.csModuleManagerTests/PatchListTest.csModuleManagerTests/PatchApplierTest.csModuleManager/ModuleManager.csprojModuleManager/PatchApplier.csModuleManagerTests/PassTest.csModuleManager/Patches/CopyPatch.cs5d77647f910ee0a35f2b0c1d3148314ff43442631524121646000https://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong@gmail.comExtract IPatch interface, split up root patches
2018-04-19 00:07:26 -07005d77647f910ee0a35f2b0c1d3148314ff4344263Extract IPatch interface, split up root patcheseditModuleManagerTests/ModuleManagerTests.csprojeditModuleManager/ModuleManager.csprojaddModuleManagerTests/Patches/PatchCompilerTest.csaddModuleManager/Patches/CopyPatch.cseditModuleManagerTests/PatchListTest.csdeleteModuleManagerTests/PatchTest.csaddModuleManager/Patches/DeletePatch.csdeleteModuleManager/Patch.cseditModuleManagerTests/PatchExtractorTest.cseditModuleManager/PatchList.csaddModuleManagerTests/Patches/EditPatchTest.cseditModuleManager/PatchApplier.cseditModuleManagerTests/PassTest.csaddModuleManagerTests/Patches/DeletePatchTest.csaddModuleManagerTests/Patches/CopyPatchTest.cseditModuleManagerTests/PatchApplierTest.cseditModuleManager/Pass.csaddModuleManager/Patches/IPatch.cseditModuleManager/PatchExtractor.csaddModuleManager/Patches/PatchCompiler.csaddModuleManager/Patches/EditPatch.cs
- ModuleManagerTests/PatchListTest.csModuleManagerTests/Patches/PatchCompilerTest.csModuleManagerTests/PassTest.csModuleManagerTests/NodeMatcherTest.csd331d9e7efd2b3c0cc1b565072492c2669e9bb261524123804000https://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong@gmail.comFix tests depending on line endings
Since we're already asserting on the ParamName, we can trust that the
2nd part of the message will be there and not test it.
2018-04-19 00:43:24 -0700d331d9e7efd2b3c0cc1b565072492c2669e9bb26Fix tests depending on line endingseditModuleManagerTests/Patches/PatchCompilerTest.cseditModuleManagerTests/NodeMatcherTest.cseditModuleManagerTests/PatchListTest.cseditModuleManagerTests/PassTest.cs
githttps://ksp.sarbian.com/jenkins/user/joeydwongjoeydwongjoeydwong