Inverted job and build functions for list_artifacts

[Sup3rGeo]

I believe that the functions list_artifacts_for_job and list_artifacts_for_build are inverted:

pybuildkite/pybuildkite/artifacts.py

Lines 19 to 42 in 5662972

	def list_artifacts_for_job(self, organization, pipeline, build):
	"""
	Returns a paginated list of a job’s artifacts.
	:param organization: organization slug
	:param pipeline: pipeline slug
	:param build: build number
	:return: Returns a paginated list of a job’s artifacts.
	"""
	url = self.path + "artifacts/"
	return self.client.get(url.format(organization, pipeline, build))
	def list_artifacts_for_build(self, organization, pipeline, build, job):
	"""
	Returns a paginated list of a build’s artifacts across all of its jobs.
	:param organization: organization slug
	:param pipeline: pipeline slug
	:param build: build number
	:param job: job id
	:return: Returns a paginated list of a build’s artifacts across all of its jobs.
	"""
	url = self.path + "jobs/{}/artifacts/"
	return self.client.get(url.format(organization, pipeline, build, job))

The list_artifacts_for_build should not need a job, only the build. The function that gets a job should be list_artifacts_for_job.

This is also in line with the API docs: https://buildkite.com/docs/apis/rest-api/artifacts

I would also be happy to open a PR for this one

[pyasi]

Nice find! And you're correct. Let's get this in (and release alongside your other PR for 0.0.6)