BREAKING CHANGE: Remove blogic option. Blogic is now always enabled by default.

[jeswr]

BREAKING CHANGE: blogic option has been removed.

[josd]

it is not a breaking change because eye still accepts --blogic but it DEPRECATED and today it is not needed as we automatically detect this logic mode.

It is commented in the code https://github.com/eyereasoner/eye/blob/beccc014842e7b473970b097796f3c5ce0299fcd/eye.pl#L920

[jeswr]

I've set it as a breaking change over here because:

1. Using the blogic option will create now create type errors here as I have removed that option from the typescript interface. So users will need to migrate by removing their use of that option.
2. blogic results are returned even when blogic: false is set (Typescript users will hit the above problem first, but plain JavaScript users won't)

Unless there is a chance of the option becoming necessary again I'm inclined to keep these breaking changes as it is a small migration path and we don't have a huge user base for blogic at the moment so only a small number of users will be impacted.

[josd]

OKOK, that fine @jeswr

[jeswr]

🎉 This PR is included in version 6.0.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release
• v6.0.0

Your semantic-release bot 📦🚀