fix(Range): Assert that number of columns is an integer

[luchinke]

When i try the following methods: sheet.get_range_from_a1("D2:FG2").get_values(), sheetfu throws the following error:
for column in range(0, self.coordinates.number_of_columns): TypeError: 'float' object cannot be interpreted as an integer

Checked further and it seems that in this case, with this range specified, the self.coordinates.number_of_columns variable contains the number '160.0'.

[philippe2803]

Happy to review.
Could you please make the tests pass before I check?

[luchinke]

I am now facing this error from the tests:

The verification check for the environment did not pass. The command "if [ "$BRANCH" == "master" ]; then semantic-release publish; fi" exited with 1.

Don't think I can fix that x)

[philippe2803]

@luchinke
@sp-daniel-sanchez

What is the status of this PR? Should we keep it or close it? As it is now, it can not be merged as the tests are not working.

[sp-daniel-sanchez]

Hi @philippe2803, gonna take a look to the semantic release issue and fix the tests asap.

[philippe2803]

I originally thought the tests actually failed, but it's only a problem with the semantic release.
Happy to merge as soon as the semantic release issue is fixed. Can someone look into it?

[SP-alex-muelas]

It was a bug in the semantic release script, it was not checking if the PR was coming from a fork. I've fixed it in this PR #39.
@luchinke can you close and re-open this PR for the travis job to run again?

[philippe2803]

LGTM.

[philippe2803]

Feel free to merge without my input.

[jlasky]

I've been using this branch for about a month now without problems. Let's get it merged!