Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Various fixes #6

Closed
wants to merge 1 commit into from
Closed

Various fixes #6

wants to merge 1 commit into from

Conversation

h3ph4est7s
Copy link

Syntax corrections,
Add support for DocString (natspec) errors,
IDE config dir ignored
fixes: #5

Add support for DocString errors,
IDE config dir ignored
@Latrasis
Copy link
Owner

Hi ppanos! Thank you for the work! I'll check this out.

@Latrasis
Copy link
Owner

Can you elaborate on the Docstring/Natspec errors?

@h3ph4est7s
Copy link
Author

h3ph4est7s commented Sep 19, 2016

The errors have to do with incomplete docstring syntax more info about the spec can be found here.
Relevant code elaborating about the errors can be found here and here searching for the appendError function calls.
Also there is a problem in the representation of docstring related errors through the compiler, it doesn't print line numbers and positions regarding the error.
I opened an issue regarding this problem here.

@Latrasis
Copy link
Owner

Latrasis commented Jun 13, 2017

Hi @h3ph4est7s, I just wanted to say I'm sincerely sorry for not following up your PR for all this time. I don't know how or why I didn't get back to you. I know it doesn't mean much at this point, but thank you for your time and work.

@Latrasis
Copy link
Owner

@h3ph4est7s: Closing this for now.

@Latrasis Latrasis closed this Jul 23, 2018
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.

Error on solidity file open
2 participants