Serialiser_Engine: Remove the 'failed' bool parameter in the deserialiser methods

[adecler]

Issues addressed by this PR

Closes #3087

Parameter removed from all methods as it wasn't used anymore and its purpose could be misunderstood.

Test files

Just make sure that all objects are still (de)serialising correctly.
@FraserGreenroyd , I'll leave it to you to decide how much testing you need on this PR. Since this PR doesn't actually change the behaviour of the serialiser, I am pretty relax about it.

Changelog

Additional comments

[adecler]

@BHoMBot check versioning

[bhombot-ci]

@adecler to confirm, the following actions are now queued:

• check versioning

[adecler]

@BHoMBot check compliance

[bhombot-ci]

@adecler to confirm, the following actions are now queued:

• check code-compliance
• check documentation-compliance
• check project-compliance
• check branch-compliance
• check dataset-compliance
• check copyright-compliance

[adecler]

@BHoMBot check serialisation

[bhombot-ci]

@adecler to confirm, the following actions are now queued:

• check serialisation

[FraserGreenroyd]

@BHoMBot check core
@BHoMBot check null-handling
@BHoMBot check unit-tests
@BHoMBot check installer

[bhombot-ci]

@FraserGreenroyd to confirm, the following actions are now queued:

• check core
• check null-handling
• check unit-tests
• check installer

There are 17 requests in the queue ahead of you.

[bhombot-ci]

FAO: @FraserGreenroyd
@FraserGreenroyd is seeking dispensation on this Pull Request to skip a required check. Please can you provide authorisation for the check to be skipped, or provide assistance as appropriate.

The check they wish to have dispensation on is unit-tests.

If you are providing dispensation on this occasion, please reply with:

@BHoMBot this is a DevOps instruction. I am authorising dispensation to be granted on check ref. 14291210363

[FraserGreenroyd]

@BHoMBot this is a DevOps instruction. I am authorising dispensation to be granted on check ref. 14291210363

[bhombot-ci]

@FraserGreenroyd I have now provided a passing check on reference 14291210363 as requested.

[FraserGreenroyd]

Code is deleted, checks pass, spot check on a couple of objects in Grasshopper, all looks good, happy to deploy.

[FraserGreenroyd]

@BHoMBot check ready-to-merge

[bhombot-ci]

@FraserGreenroyd to confirm, the following actions are now queued:

• check ready-to-merge