Open epost opened 4 years ago
Should the Operator
type also be in Statebox.Core
? See https://github.com/statebox/purescript-studio/pull/326/commits/b591ca55ad13b1319276f65636765cfb336a3987
I added this to the checklist to look into, thanks @sjoerdvisscher.
goals
tasks
Path
should be defined inStatebox.Core.WiringTree
. (See also Marco's executions PR #334 for work on this.)Wiring.Tree
or whateverlabels
inWiring
; is nowArray Int
, should beArray <some newtype>
Wiring
newtype vs.WiringRaw
Statebox.Core.Net
fromStatebox.Core.Types
Statebox.Core.Diagram
fromStatebox.Core.Types
Statebox.Core.Wiring
fromStatebox.Core.Types
labels
newtype mentioned in parent task.Statebox.Core.Firing
fromStatebox.Core.Types
Statebox.Core.Initial
fromStatebox.Core.Types
Statebox.Core.Types
Statebox.Core.Types
into submodules, and re-export them.Statebox.Core.Types
imports; useStatebox.Core
instead?Statebox.Core.Types
module altogether?Singleton
should go into `Statebox.Core.Common.Message
should go into `Statebox.Core.Common.TxId
should go intoStatebox.Core.Common
.TxId
should be removed fromStatebox.Transaction
(duplication).HexStr
should go intoStatebox.Core.Common
.HexStr
should be removed elsewhere.HashStr
(should go intoStatebox.Core.Common
instead ofStatebox.Transaction
).stbx-protocol/src/Statebox/Protocol/Fire.purs
doesn't have the duplicatedMarking
stuff (see here).Tokens
,TransitionMarkingF
,PlaceMarkingF
are not duplicated (see here).stbx-core
underStatebox
orStatebox.Core
look forInts
,type
aliases, etc.Route
s should be implemented in terms of these.findXXX
methods inStudio.Model
should be split off and factored out into their own module;Transaction.Cache
perhaps?Operator
type also be inStatebox.Core
? See https://github.com/statebox/purescript-studio/pull/326/commits/b591ca55ad13b1319276f65636765cfb336a3987