I find the naming for the method symbol not to be appropriate, i hope you do not find that rude. It doesn't communicate what the method actually does. I would suggest that it be renamed to something from this:
NseFinance.symbol('OANDO')
to this
NseFinance.get_company_info('OANDO')
I have already gone ahead to make that change on my fork locally. If you agree, i can send a pull request for it to be merged into master.
I find the naming for the method symbol not to be appropriate, i hope you do not find that rude. It doesn't communicate what the method actually does. I would suggest that it be renamed to something from this:
to this
I have already gone ahead to make that change on my fork locally. If you agree, i can send a pull request for it to be merged into master.