Uh oh!
There was an error while loading. Please reload this page.
Allow Multi-Platform, starting with Fabric - #8
Conversation
RhythmicSys
left a comment
There was a problem hiding this comment.
I don't think I had any code changes I saw that were needed, just general directional ideas
There was a problem hiding this comment.
Would eventually like to extract the commands to common, since they use brig
There was a problem hiding this comment.
Would like to move this into common along with commands
There was a problem hiding this comment.
Would also like to move this into common, unsure if other platforms will need separate types of exceptions and handling
There was a problem hiding this comment.
I am unsure how the other platforms handle permissions, this might be able to be changed and made a common class, or converted into some sort of enum with wrappers for the different platforms
Uh oh!
There was an error while loading. Please reload this page.
Created modules for paper and fabric platforms Paper is functional on maven package Code is not refactored to common yet, common empty
97d345e to
2502edeCompare
Warning
This PR is incomplete and should not be merged until final code review is completed.
This PR is to enable multi-platform support for SimplePMs. It is likely we will not merge until Fabric is implemented, but this is unclear right now.
Steps