Skip to content

Remove snapshot content from rspec description#49

Merged
levinmr merged 2 commits into
levinmr:masterfrom
dmorgan-fa:provide-snapshot-path-in-rspec-description
Apr 3, 2026
Merged

Remove snapshot content from rspec description#49
levinmr merged 2 commits into
levinmr:masterfrom
dmorgan-fa:provide-snapshot-path-in-rspec-description

Conversation

@dmorgan-fa

Copy link
Copy Markdown
Contributor

What

Provide the snapshot file path rather than the contents of the snapshot in rspec description.

The snapshot can be many hundreds/thousands of lines long, and doesn't provide much benefit as the description.

# What
Provide the snapshot file path rather than the contents of the snapshot in rspec description.

The snapshot can be many hundreds/thousands of lines long, and doesn't provide much benefit as the description.

- resolves: #48
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

lukearndt added a commit to lukearndt/rspec-snapshot that referenced this pull request Oct 24, 2025
[Cloned from levinmr#49](levinmr#49)

Provide the snapshot file path rather than the contents of the snapshot in rspec description.

The snapshot can be many hundreds/thousands of lines long, and doesn't provide much benefit as the description.

- resolves: levinmr#48
@sonarqubecloud

sonarqubecloud Bot commented Apr 3, 2026

Copy link
Copy Markdown

@levinmr levinmr merged commit c6de007 into levinmr:master Apr 3, 2026
8 checks passed
@dmorgan-fa dmorgan-fa deleted the provide-snapshot-path-in-rspec-description branch April 6, 2026 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Request: Remove the snapshot content from the rspec description

2 participants