Skip to content
This repository was archived by the owner on Sep 30, 2021. It is now read-only.

Improving variable name. #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MisterAwesome23
Copy link

An improved variable name represents what information the variable contains in a more exact sense.

An improved variable name represents what information the variable contains in a more exact sense.
@thekunalsaini
Copy link

Hi, @MisterAwesome23
As mentioned in comment what is its purpose then why to change name?

@MisterAwesome23
Copy link
Author

Thingspeak has two API keys to start up front.
Read API key and write API key.

Now it might be confusing whether the "read" is to read from device or read from ThingsSpeak (at least was confusing to me at first)

So, writing an exact variable ensures which KEY has to be imported from ThingsSpeak removing an ambiguity.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants