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

Reorder additional test deps so that the given deps are placed first #18

Merged
merged 2 commits into from
Feb 23, 2022

Conversation

minkuan88
Copy link
Contributor

No description provided.

@arunkumar9t2
Copy link
Contributor

LGTM, nice find. Please add a test.

@arunkumar9t2 arunkumar9t2 added the bug Something isn't working label Feb 23, 2022
@minkuan88
Copy link
Contributor Author

LGTM, nice find. Please add a test.

Test added in the additional commit

@arunkumar9t2 arunkumar9t2 merged commit 001bd5b into master Feb 23, 2022
@arunkumar9t2 arunkumar9t2 deleted the reorder-additional-test-deps branch February 23, 2022 08:12
akshay-grabtaxi pushed a commit that referenced this pull request Jun 28, 2023
…18)

* Reorder additional test deps so that the given deps are placed first

* Add test to ensure given deps for test are prioritised
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants