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

test: add tests for our changes to libzip #4736

Merged
merged 3 commits into from
Aug 18, 2022
Merged

Conversation

merceyz
Copy link
Member

@merceyz merceyz commented Aug 11, 2022

What's the problem this PR addresses?

We're missing tests for the changes in our forked libzip.

How did you fix it?

Add tests.

Checklist

  • I have read the Contributing Guide.
  • I have set the packages that need to be released for my changes to be effective.
  • I will check that all automated PR checks pass before the PR gets reviewed.

@merceyz merceyz changed the title Merceyz/test/libzip test: add tests for our changes to libzip Aug 11, 2022
@merceyz merceyz force-pushed the merceyz/test/libzip branch from 723834b to ed072af Compare August 12, 2022 15:57
@merceyz merceyz marked this pull request as ready for review August 12, 2022 15:58
@merceyz merceyz force-pushed the merceyz/test/libzip branch from ed072af to 1e40ae9 Compare August 12, 2022 16:00
@arcanis arcanis merged commit 5e64622 into master Aug 18, 2022
@arcanis arcanis deleted the merceyz/test/libzip branch August 18, 2022 09:37
merceyz added a commit that referenced this pull request Oct 21, 2023
* test: mtime can be updated

* test: timezone is ignored

* test: DST is ignored

(cherry picked from commit 5e64622)
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.

2 participants