Update supported Python versions and update CI/CD

[diegoferigo]

No description provided.

[flferretti]

Thanks a lot Diego! LGTM, I left only a couple of minor comments

[flferretti]

If the environment variable can only be ROD_LOGGING_LEVEL, is there a reason why you're passing it as the argument of this function? Can it just be hardcoded?

Suggested change

	def get_default_logging_level(env_var: str) -> logging.LoggingLevel:
	"""
	Get the default logging level.
	Args:
	env_var: The environment variable to check.
	Returns:
	The logging level to set.
	"""
	import os
	try:
	return logging.LoggingLevel[os.environ[env_var].upper()]
	# Raise if the environment variable is set but the logging level is invalid.
	except AttributeError:
	msg = f"Invalid logging level defined in {env_var}: '{os.environ[env_var]}'"
	raise ValueError(msg)
	# If the environment variable is not set, return the logging level depending
	# on the installation mode.
	except KeyError:
	return (
	logging.LoggingLevel.DEBUG
	if installation_is_editable()
	else logging.LoggingLevel.WARNING
	)
	# Configure the logger with the default logging level.
	logging.configure(level=get_default_logging_level(env_var="ROD_LOGGING_LEVEL"))
	def get_default_logging_level() -> logging.LoggingLevel:
	"""
	Get the default logging level.
	Returns:
	The logging level to set.
	"""
	import os
	try:
	return logging.LoggingLevel[os.environ["ROD_LOGGING_LEVEL"].upper()]
	# Raise if the environment variable is set but the logging level is invalid.
	except AttributeError:
	msg = f"Invalid logging level defined in 'ROD_LOGGING_LEVEL'"
	raise ValueError(msg)
	# If the environment variable is not set, return the logging level depending
	# on the installation mode.
	except KeyError:
	return (
	logging.LoggingLevel.DEBUG
	if installation_is_editable()
	else logging.LoggingLevel.WARNING
	)
	# Configure the logger with the default logging level.
	logging.configure(level=get_default_logging_level()

[diegoferigo]

Having the environment variable explicitly defined makes it more visible to users rather than hiding it inside the logic of a function. Since we do not have documentation, I valued readability.

[flferretti]

It makes sense, thanks for the explanation!

[diegoferigo]

FYI @lorycontixd I had to update your test introduced in #33 to make it work on Windows.

[lorycontixd]

@diegoferigo Yep, no problem.