Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Snapshot tests for scarb-doc #1420

Merged
merged 5 commits into from
Jul 8, 2024
Merged

Snapshot tests for scarb-doc #1420

merged 5 commits into from
Jul 8, 2024

Conversation

piotmag769
Copy link
Contributor

@piotmag769 piotmag769 commented Jul 5, 2024

Closes #1377
UPDATE_EXPECT=1 cargo test fixed the tests (changes the file content)

@piotmag769 piotmag769 requested review from mkaput and Arcticae and removed request for THenry14 and DelevoXDG July 5, 2024 12:17
Cargo.toml Outdated Show resolved Hide resolved
@maciektr
Copy link
Contributor

maciektr commented Jul 5, 2024

UPDATE_EXPECT=1

Can you doc this somewhere in code? It's not really something that I would EXPECT :D

Other than that, nothing to review by me.

Cargo.lock Outdated Show resolved Hide resolved
@piotmag769 piotmag769 requested a review from mkaput July 6, 2024 10:42
@piotmag769 piotmag769 enabled auto-merge July 8, 2024 14:07
@piotmag769 piotmag769 added this pull request to the merge queue Jul 8, 2024
Merged via the queue into main with commit 3c1bf2c Jul 8, 2024
22 checks passed
@piotmag769 piotmag769 deleted the scarb-doc/snapshot-tests branch July 8, 2024 15:04
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.

Add snapshot-based tests
4 participants