Remove duplicate logic

[parkerbxyz]

Proposed Changes

I found some logic in issue_metrics.py that is duplicating environment variable processing already happening via get_env_vars() in config.py, which was introduced in #171.

issue-metrics/config.py

Lines 61 to 71 in ebc6a99

	labels_to_measure = os.getenv("LABELS_TO_MEASURE")
	if labels_to_measure:
	labels_to_measure = labels_to_measure.split(",")
	else:
	labels_to_measure = []
	ignore_users = os.getenv("IGNORE_USERS")
	if ignore_users:
	ignore_users = ignore_users.split(",")
	else:
	ignore_users = []

Duplicate logic:

issue-metrics/issue_metrics.py

Lines 260 to 264 in ebc6a99

	labels = env_vars.labels_to_measure
	if labels:
	labels = labels.split(",")
	else:
	labels = []

issue-metrics/issue_metrics.py

Lines 140 to 141 in ebc6a99

	if ignore_users is None:
	ignore_users = []

Removing this duplicate logic fixes #180.

Readiness Checklist

Author/Contributor

• If documentation is needed for this change, has that been included in this pull request
• run make lint and fix any issues that you have introduced
• run make test and ensure you have test coverage for the lines you are introducing

Reviewer

• Label as either bug, documentation, enhancement, infrastructure, or breaking

[zkoppert]

Thanks for the contribution! good find!