Skip to content

Update filtering in order to remove any waiting time#53

Merged
hurricane642 merged 2 commits intomainfrom
filter_yaml_speedup
Nov 8, 2023
Merged

Update filtering in order to remove any waiting time#53
hurricane642 merged 2 commits intomainfrom
filter_yaml_speedup

Conversation

@hurricane642
Copy link
Copy Markdown
Collaborator

Continuation of the Issue #12 solution.
After discussion in #52 , it was decided to change the file structure. All tags will now be specified in the main file test_listing.yml. Accordingly, the functions of getting the full list of tests, getting test tags and filtering them were rewritten. As a result, now the filtering of tests is almost instantaneous.
Next, we will need to update the tests in Perturbo according to the changed structure, as well as update all Docker Images. Beyond that, I propose to do a new minor release.

@hurricane642 hurricane642 added high priority Important task (should solve it as soon as possible) usability improve experience of package usage labels Nov 8, 2023
@hurricane642 hurricane642 requested a review from imaliyov November 8, 2023 00:19
@hurricane642 hurricane642 self-assigned this Nov 8, 2023
@imaliyov
Copy link
Copy Markdown
Member

imaliyov commented Nov 8, 2023

It's great to hear that the filtering is fast now!!
I don't see the test_listing.yml among the modified files in the commits though...

@hurricane642
Copy link
Copy Markdown
Collaborator Author

It's great to hear that the filtering is fast now!! I don't see the test_listing.yml among the modified files in the commits though...

Yes, it's because this file is saved in the perturbo repo with the other tests files.

@hurricane642 hurricane642 merged commit 56b51b1 into main Nov 8, 2023
@hurricane642 hurricane642 deleted the filter_yaml_speedup branch November 8, 2023 21:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

high priority Important task (should solve it as soon as possible) usability improve experience of package usage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants