Skip to content

Conversation

@osfameron
Copy link
Collaborator

Contrived example just to show some output.

@brett19 Could you sanity check this / advise how to make this a bit more realistic/useful?

Contrived example just to show some output.
@osfameron
Copy link
Collaborator Author

Adding @maria-robobug to PR to try to close this one if mildly useful

@osfameron osfameron marked this pull request as ready for review January 24, 2022 12:43
Copy link
Contributor

@maria-robobug maria-robobug left a comment

Choose a reason for hiding this comment

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

👍 LGTM, but just needs a test added

@@ -0,0 +1,46 @@
"use strict";

Copy link
Contributor

Choose a reason for hiding this comment

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

Just wondering are we using this example anywhere? Or is this prep for referencing later?

Also this will need a new test.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

hmm yes, looks like this needs some more work, ta

@osfameron osfameron marked this pull request as draft January 25, 2022 16:36
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