Skip to content

Feature/combine test durations #91

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Feature/combine test durations #91

wants to merge 5 commits into from

Conversation

joaofogoncalves
Copy link

@joaofogoncalves joaofogoncalves commented Apr 13, 2024 •
edited
Loading

Description

Adding a cli command do combine several test durations files into one single file.
Useful when running parallel jobs in GH actions and after want to combine all changes into a single file.

Checklist

  • Tests covering the new functionality have been added
  • Documentation has been updated OR the changes are too minor to be documented
  • The Changes are listed in the CHANGELOG.md OR the changes are insignificant

@jerry-git
Copy link
Owner

Great, thank you! Could you add some test(s) and make the CI green.

@joaofogoncalves
Adding root folder argument and fixing CD (#2) (#3)
45b5053 
* Update CHANGELOG.md

* Adding root folder argument and fixing CD

* Adding tests

* Update test_cli.py
@joaofogoncalves
Copy link
Author

Great, thank you! Could you add some test(s) and make the CI green.

I Added tests and fixed the failing pre-commit checks.

@agriyakhetarpal agriyakhetarpal mentioned this pull request Oct 22, 2024
Merged
@eiriklid
Copy link

@jerry-git is this something I can help out with?
Would like to have this feature as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jerry-git jerry-git Awaiting requested review from jerry-git jerry-git is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@joaofogoncalves @jerry-git @eiriklid