Skip to content

Refactor issue list and pr list tests#6135

Merged
vilmibm merged 1 commit intocli:trunkfrom
satotake:inject-now-to-test
Aug 24, 2022
Merged

Refactor issue list and pr list tests#6135
vilmibm merged 1 commit intocli:trunkfrom
satotake:inject-now-to-test

Conversation

@satotake
Copy link
Contributor

following #6085

  • Inject fake Now function to tests of issue list / pr list
  • Remove regexp from the tests

following cli#6085

* Inject fake `Now` function to tests of `issue list` / `pr list`
* Remove `regexp` from the tests
@satotake satotake requested a review from a team as a code owner August 24, 2022 15:04
@satotake satotake requested review from samcoe and removed request for a team August 24, 2022 15:04
@cliAutomation cliAutomation added the external pull request originating outside of the CLI core team label Aug 24, 2022
Copy link
Contributor

@vilmibm vilmibm left a comment

Choose a reason for hiding this comment

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

thank you!!

@vilmibm vilmibm merged commit f8d7c1c into cli:trunk Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external pull request originating outside of the CLI core team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants