before_script needs to be an array

[baurmatt]

Purpose

Fix Gitlab CI pipeline

jobs:mamba:before_script config should be an array containing strings and arrays of strings

Approach

Converting the YAML object type to an array

Open Questions and Pre-Merge TODOs

• Use github checklists. When solved, check the box and explain the answer.

Resources & Links

Links to blog posts, patterns, libraries or addons used to solve this problem

[coveralls]

Coverage remained the same at 97.835% when pulling 05aead8 on baurmatt:gitlab-ci-before-script into 5869402 on pyscaffold:master.

[abravalheri]

Hi @baurmatt, thank you very much for the PR.

Can I ask you how you are getting this error? Is this an specific linting tool?

I have tested the pipeline definition with the before_script being a string instead of an array and everything seems to be running fine as shown here: https://gitlab.com/pyscaffold/ci-tester/-/jobs/2193180299.
(The pipeline configuration can be found here: https://gitlab.com/pyscaffold/ci-tester/-/blob/2a65db816b6bbba146e0fe7338ad5cc389701676/.gitlab-ci.yml)

I also cannot find any error with the CI editor/lint: https://docs.gitlab.com/ee/ci/pipeline_editor/index.html

[baurmatt]

@abravalheri This was a CI job running on a self hosted Gitlab Enterprise 14.7.4. Interesting that it works on gitlab.com 🤔

[abravalheri]

Might be a version thing...

If that is the case, maybe the best is to use arrays for all the scripts, before_scripts, after_scripts, etc right? What do you think?

Would you be interesting in changing the other places as well?

[baurmatt]

Might be a version thing...

Probably, I have no explanation for it oO

Might be a version thing...

Sounds good!

Would you be interesting in changing the other places as well?

Sure thing! Done :)

[abravalheri]

Amazing!

Thank you very much for all the help @baurmatt. I will merge this change, run some tests in our ci-tester repository on GitLab (just as a sanity check) and then release a patch version shortly.