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.

Sign up for GitHub

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

Jump to bottom

Remove result.getValueAt() ; replace usage with result.getValuesAt() #100

Open
critcortex opened this issue Sep 13, 2017 · 0 comments
Open

Remove result.getValueAt() ; replace usage with result.getValuesAt() #100

critcortex opened this issue Sep 13, 2017 · 0 comments
Assignees
@davidkedz
Labels
Priority 1

Comments

@critcortex
Copy link
Contributor

critcortex commented Sep 13, 2017 •
edited
Loading

Currently we have two similar methods that perform approximately the same functionality, but have slightly different usages and different function signatures.

For better modularity, these should be merged, with result.getValuesAt() replacing result.getValueAt()
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@davidkedz davidkedz

Labels
Priority 1
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@critcortex @davidkedz