Skip to content

Update regression test for issue #835.#1417

Merged
RyanGlScott merged 1 commit intoGaloisInc:masterfrom
taimoorzaeem:issue1329
Aug 26, 2022
Merged

Update regression test for issue #835.#1417
RyanGlScott merged 1 commit intoGaloisInc:masterfrom
taimoorzaeem:issue1329

Conversation

@taimoorzaeem
Copy link
Contributor

Fixes issue #1329.

@RyanGlScott
Copy link
Contributor

Thanks! By the way, for future reference: you don't need to open a new PR for each new change you make. Feel free to rebase your changes onto the same branch. This helps keeps the discussion for a particular task contained within a single PR, which makes it easier to review.

@taimoorzaeem
Copy link
Contributor Author

This helps keeps the discussion for a particular task contained within a single PR, which

Yeah sure!

Copy link
Contributor

@RyanGlScott RyanGlScott left a comment

Choose a reason for hiding this comment

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

Happily, the CI passes this time. Let's land this!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants