Skip to content

Conversation

@wleczny
Copy link
Contributor

@wleczny wleczny commented Jul 28, 2022

No description provided.

@wleczny wleczny requested a review from lwronski July 28, 2022 21:20
@wleczny wleczny linked an issue Jul 29, 2022 that may be closed by this pull request
Copy link
Contributor

@lwronski lwronski left a comment

Choose a reason for hiding this comment

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

LGTM, I suggest only to add a documentation.

@wleczny wleczny force-pushed the output-option-for-using-directives branch 3 times, most recently from 2d111ad to 5d94ea7 Compare August 1, 2022 08:18
@wleczny wleczny marked this pull request as ready for review August 1, 2022 08:19
@wleczny wleczny added the enhancement New feature or request label Aug 1, 2022
@wleczny wleczny requested review from Gedochao and lwronski August 1, 2022 08:33
Copy link
Contributor

@Gedochao Gedochao left a comment

Choose a reason for hiding this comment

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

Left some language nitpick comments on the docs, otherwise LGTM

@wleczny wleczny force-pushed the output-option-for-using-directives branch from 5d94ea7 to 7f9b020 Compare August 1, 2022 09:01
@wleczny wleczny merged commit cad549f into VirtusLab:main Aug 1, 2022
@wleczny wleczny deleted the output-option-for-using-directives branch August 1, 2022 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

4 participants