Add dependencies to dune-project file

[Nadrieril]

@giltho like this?

[giltho]

(menhir :build) is needed in name_matcher_parser's package definition

[giltho]

After installing menhir that seems to be enough :)
Happy to do one last check from fresh switch before you merge if needed

[Nadrieril]

(menhir :build) is needed in name_matcher_parser's package definition

Done, would you mind one last check?

[giltho]

Ah sorry I should have been clearer, (menhir :build) is a dependency requirement, it means menhir is needed as a build dependency. So it needs to be in the list of dependencies:

(package
  (name name_matcher_parser)
  (synopsis "Parser to define name matchers")
  (description "")
  (depends
    (menhir :build)
    menhirLib
    ppx_deriving
    visitors
    zarith
  )
 )

[giltho]

I'm sorry, I should have just fixed it myself...
opam is weird like this, it's so difficult to get these things right it's annoying.

[Nadrieril]

how about now?

[giltho]

Opam files need to be regenerated 😭

[Nadrieril]

o no x)

[Nadrieril]

And now?

[giltho]

Success!! Thank you!

[Nadrieril]

Hooray! Thanks for walking me through this 😁