feat: remove snap suffix if use custom identifier

kibala145 commented 1 year ago

Description

When using customSnapshotIdentifier do not add -snap suffix

It would be great not to force '-snap' suffix so users are able to customize file name providing customSnapshotIdentifier There are some discussion about this https://github.com/americanexpress/jest-image-snapshot/issues/107

Current test cases passed. Some of them corrected.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (adding or updating documentation)
[ ] Dependency update

[ ] My change requires a change to the documentation and I have updated the documentation accordingly.
[x] My changes are in sync with the code style of this project.
[x] There aren't any other open Pull Requests for the same issue/update.
[ ] These changes should be applied to a maintenance branch.
[ ] I have added the Apache 2.0 license header to any new files created.

Better screenshots naming

CLAassistant commented 1 year ago

All committers have signed the CLA.

kibala145 commented 1 year ago

Resolved conflicts locally and pushed with force so licence/cla check passed. There are no new changes.

oneamexbot commented 1 year ago

:tada: This PR is included in version 5.2.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: