Silenced failures on yaml parsing with an empty changeset

[brymacto]

paper_trail/lib/paper_trail/version_concern.rb

Line 359 in eb58b54

::Kernel.warn format(E_YAML_PERMITTED_CLASSES, e)

When a yaml parsing error is encountered, an empty object changes object is returned with a warning logged.

Is it not better to fail fast here and let an exception surface? The empty object is misleading and difficult to debug.

If there's openness to such a change, I could open a PR.

[gurgelrenan]

@brymacto Could you reproduce this issue following this guide? https://github.com/paper-trail-gem/paper_trail/blob/master/.github/ISSUE_TEMPLATE/bug-report.md

[github-actions]

This issue has been automatically marked as stale due to inactivity.
The resources of our volunteers are limited.
Bug reports must provide a script that reproduces the bug, using our template. Feature suggestions must include a promise to build the feature yourself.
Thank you for all your contributions.