[Notes] [Git][BuildStream/buildstream][master] 2 commits: .gitlab-ci.yml: Trying to fix code quality reports

[Tristan Van Berkom <gitlab mg gitlab com>]

Title: GitLab

Tristan Van Berkom pushed to branch master at BuildStream / buildstream

Commits:

• 5fb772b5
  by Tristan Van Berkom at 2018-08-26T07:30:58Z

  .gitlab-ci.yml: Trying to fix code quality reports
  
  This hasnt been working for a while, it might be because GitLab
  doesnt quite grasp that we have multiple branches to merge MRs to,
  or it might be because they have change the API and broken back compat
  in some way.
  
  See: https://docs.gitlab.com/ee/ci/examples/code_quality.html
  

• fa696307
  by Tristan Van Berkom at 2018-08-26T08:10:59Z

  Merge branch 'tristan/fix-code-quality' into 'master'
  
  .gitlab-ci.yml: Trying to fix code quality reports
  
  See merge request BuildStream/buildstream!735

1 changed file:

• .gitlab-ci.yml

Changes:

.gitlab-ci.yml

---

...	...	@@ -182,11 +182,9 @@ docs:
182	182	# Post stage #
183	183	#####################################################
184	184
185		-# Check code quality with codeclimate
186		-# This needs some refactoring; we probably just want to provide the codeclimate.json directly
187		-# as an output of radon, with some conversion
	185	+# Check code quality with gitlab's built-in feature.
188	186	#
189		-codequality:
	187	+code_quality:
190	188	<<: *tests-condition
191	189
192	190	image: docker:stable
...	...	@@ -204,7 +202,7 @@ codequality:
204	202	--volume /var/run/docker.sock:/var/run/docker.sock
205	203	"registry.gitlab.com/gitlab-org/security-products/codequality:$SP_VERSION" /code
206	204	artifacts:
207		- paths: [codeclimate.json]
	205	+ paths: [gl-code-quality-report.json]
208	206
209	207	analysis:
210	208	<<: *tests-condition

—
View it on GitLab.
You're receiving this email because of your account on gitlab.com. If you'd like to receive fewer emails, you can adjust your notification settings.