Improve variable naming.

[HarshKapadia2]

Nice Action!

In my opinion, variables like prbody, prurl and PRNUM would be more readable if they had underscores (_) or hyphens (-) to separate them.

prurl ➡️ pr-url (underscore didn't look good to me here)
prbody ➡️ pr-body (underscore didn't look good to me here)
PRNUM ➡️ PR_NUM (hyphen didn't look good to me here)

Cc: @tusharnankani

[HarshCasper]

Hi @HarshKapadia2 @tusharnankani 👋

Thanks for the suggestions! Would you like to submit a PR for any of this? I would be obliged for the purpose 😄

[HarshKapadia2]

Sure @HarshCasper, I wouldn't mind submitting a PR. I'll do it on the 17th if that's okay with you, as I have end semester exams going on. Sorry!

[HarshCasper]

Sure @HarshCasper, I wouldn't mind submitting a PR. I'll do it on the 17th if that's okay with you, as I have end semester exams going on. Sorry!

Please take your time! Best of luck with your exams :)

[HarshKapadia2]

Hey @HarshCasper I was working on this issue, but I had a doubt. Where are the following environment variables being set?

validate-issues-over-pull-requests/main.py

Lines 15 to 16 in 1b3d410

	body = os.getenv("INPUT_PRBODY")
	url = os.getenv("INPUT_PRURL")

I'm asking because I want to rename them to INPUT_PR_BODY and INPUT_PR_URL, but wasn't sure where they're being set. It might cause errors if I rename them without knowing how they're set. I couldn't find anything in the docs. Maybe I missed it.

[HarshKapadia2]

Hey @HarshCasper any updates?

[HarshCasper]

Hi @HarshKapadia2 — Apologies for the late response.

Here are the references:

validate-issues-over-pull-requests/action.yml

Line 8 in 1b3d410

inputs:

validate-issues-over-pull-requests/main.py

Line 15 in 1b3d410

body = os.getenv("INPUT_PRBODY")

I hope it makes sense now. Please let me know if you have any further questions and I will respond over quickly (subscribed to this conversation).

[Ismoh]

Any news on this? @HarshKapadia2