Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

moodle-plugin-ci on GitLab CI (extended) #36

Closed
wants to merge 3 commits into from

Conversation

Dagefoerde
Copy link
Contributor

Hi, I have applied the awesome idea of @danielneis (#15) to my own projects and found it very useful. I have modified the config in the following ways:

  • Enable matrix builds by specifying a default job template (job_template) from which builds can inherit. Builds can then specify their respective PHP versions and Moodle branches.
  • Created a symbolic link node to the nodejs binary, since csslint expects to be able to call node.
  • Removed the phpenv to remove xdebug, as it did not work out for me.

Sorry for creating an additional pull request; I was unable to modify @danielneis's branch myself. Branched from his commit, though :)

@polothy
Copy link
Contributor

polothy commented Feb 6, 2017

Should have commented sooner, but supporting other CI systems is out of scope for this project. Thank you for the time you spent on this.

I'd be happy to create a doc page with a link to something like a GitHub Gist with the YAML for GitLab. Then folks can collaborate there on it.

@polothy polothy closed this Feb 6, 2017
@Dagefoerde
Copy link
Contributor Author

Sure, I do not require official support, as it is running perfectly fine right now :)

I liked your idea of creating a gist for the YAML file, so I pasted this pull request's file here: https://gist.github.com/Dagefoerde/b8ebf54438b8a77ee4ea73f552fc0a01 . I guess, others would appreciate a link to this gist. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants