Skip to content

Conversation

@gethari
Copy link

@gethari gethari commented Oct 17, 2019

Based on this discussion , i have made some changes.

Sorry if my VS settings overridden any code formatting

@Keboo Keboo self-requested a review October 17, 2019 16:30
@Keboo
Copy link
Owner

Keboo commented Oct 25, 2019

@gethari sorry for the long delay.
Take a look at the changes that I pushed to your branch. We are not trying to log the position in the AutoDI course where the errors occurred, instead we are trying to log the position in the source code that AutoDI acts on. That way when these errors/warnings show up in the VS output window the user can simply click on them to navigate.

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