update "Fix #253: Use AdcStart() before each ReadAnalog value"

[dhineshkumarmcci]

Advanced the version number in README.md and CatenaBase.h

[terrillmoore]

This is not part of v0.19.0. If someone checks out v0.19.0 (the tag), they won't get this change.

You need to move line 1581 AHEAD of 1381, and not change 1384. See comments.

[dhineshkumarmcci]

@terrillmoore changes in the file CatenaStm32L0_ReadAnalog.cpp of the branch issue253 is not reflected in the master during the merge of this pull-request.

Catena-Arduino-Platform/src/lib/stm32/stm32l0/CatenaStm32L0_ReadAnalog.cpp

Lines 174 to 177 in 88a771a

	{
	AdcStart();
	fStatusOk = AdcGetValue(&vChannel);
	}

Trying to create a new pull-request with branch issue253 says no changes to compare. Is that ok to create a new branch and submit a pull-request?

[terrillmoore]

It would be best to understand how this happened. I suggest that you fork this repo, first apply your pull request there, then you can push changes upstream from your repo once you are sure they are correct.

[terrillmoore]

I think the merge didn't pick up your changes -- you committed changes to issue253 after I merged. Reopen #253, rename your local branch to "issue253a" or something, and submit a new pr.

[terrillmoore]

Note this from the history above:

The highlighted commits appear after the [Merged] indication. As far as I know, there is no way to merge the extra commits without a new PR. They don't point to the reference lines.

More research points to #254. I reverted that, because the documentation was not done.

Proper process was not followed. Proper process is to push additional changes to the branch used for PR #254, not to create a new branch. I am going to REVERT this PR as well, and you must push ALL your changes onto either #254, or a different new PR. Please get review from @mcciyssaroha for this, thanks.