Skip to content

Speed up test collection#298

Merged
mkannwischer merged 5 commits intomasterfrom
generate-tests
Jun 22, 2020
Merged

Speed up test collection#298
mkannwischer merged 5 commits intomasterfrom
generate-tests

Conversation

@thomwiggers
Copy link
Copy Markdown
Member

@thomwiggers thomwiggers commented May 29, 2020 •
edited
Loading

By caching and postponing operations that touch the file system or the metadata, this greatly speeds up test collection.

@thomwiggers thomwiggers changed the title <!-- This template will help you get your code into PQClean. --> Speed up test collection May 29, 2020
@thomwiggers thomwiggers added CI Issues that have to do with testing and continuous integration enhancement New feature or request labels May 29, 2020
@thomwiggers thomwiggers requested a review from mkannwischer June 12, 2020 10:56
@thomwiggers thomwiggers added the ready for review Pull request that is ready to be reviewed and merged label Jun 12, 2020
mkannwischer
mkannwischer approved these changes Jun 22, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@mkannwischer mkannwischer left a comment

Choose a reason for hiding this comment

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

LGTM

@mkannwischer mkannwischer merged commit 9e4d07d into master Jun 22, 2020
@mkannwischer mkannwischer deleted the generate-tests branch June 22, 2020 02:10
@thomwiggers thomwiggers mentioned this pull request Jun 22, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mkannwischer mkannwischer mkannwischer approved these changes

Assignees

No one assigned

Labels

CI Issues that have to do with testing and continuous integration enhancement New feature or request ready for review Pull request that is ready to be reviewed and merged

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@thomwiggers @mkannwischer