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

Update playTests.sh #2927

Closed
wants to merge 5 commits into from
Closed

Update playTests.sh #2927

wants to merge 5 commits into from

Conversation

15596858998
Copy link
Contributor

add test case

@Cyan4973
Copy link
Contributor

@terrelln : does that PR fits well with your current refactoring of the test suite ?

Copy link
Contributor

@Cyan4973 Cyan4973 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure why this PR isn't merged yet,
but it still looks like a good contribution,

@Cyan4973 Cyan4973 assigned Cyan4973 and unassigned terrelln Mar 12, 2024
@Cyan4973
Copy link
Contributor

I'll take care of it.
The tests proposed here are a good idea, but they need to be modified a bit before integration.

Cyan4973 added a commit that referenced this pull request Mar 12, 2024
centered around -T# and --fast=# arguments
Cyan4973 added a commit that referenced this pull request Mar 12, 2024
@Cyan4973
Copy link
Contributor

superseded by #3948

@Cyan4973 Cyan4973 closed this Mar 12, 2024
hswong3i pushed a commit to alvistack/facebook-zstd that referenced this pull request Mar 27, 2024
centered around -T# and --fast=# arguments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants