Add gitlab CI

Prashant.c → made their first commit to this issue’s fork.

Incorporated a .gitlab-ci.yml file into the project's root directory by utilizing the Drupal Association's template.gitlab-ci.yml contents. Continuously learning about it as I progress.

Thanks

vipin.mittal18 → made their first commit to this issue’s fork.

@Prashant.c Thanks for adding this.

@vipin.mittal18 It looks like you fixed some coding standards and PHPStan issues. However, I think that is out of scope for this MR. This is specifically about adding GitLab CI, not fixing the problems identified by GitLab CI, which should be separate issues.

Adding the CI template is very easy if that is the only file in the commit to review.

Hello @ptmkenny, I have modified gitlabci.yml file only. Refer https://git.drupalcode.org/project/schema_metatag/-/merge_requests/49/di.... Coding standards are previously committed by someone.

There are still irrelevant changes not related to the GitLab CI template.

Also, it seems that the comments have been stripped from the latest version of the template (https://git.drupalcode.org/project/gitlab_templates/-/blob/main/gitlab-c...), but I think the comments provide useful information.

I've raised MR!55 against 3.0.x-dev branch please review and merge this one if possible.

RTBC

This needs work.

@poorva If you mark RTBC, you need to state why.

Hi @ptmkenny,
Can you please review MR!55: https://git.drupalcode.org/project/schema_metatag/-/merge_requests/55/diffs and if it possible can you merge this one?

@ankitv18 Ok, I reviewed MR55. This looks better since it adds the template and leaves the tests as a separate issue, but I think the links to the documentation should be restored.

@ptmkenny I've updated the MR!55 https://git.drupalcode.org/project/schema_metatag/-/merge_requests/55/diffs , please review and check whether this one can be merged to 3.0.x-dev

@Rajeshreeputra Your addition of the previous major test needs additional configuration since this module requires PHP 8.

ankitv18 → changed the visibility of the branch 3408108-add-gitlab-ci to hidden.

This looks good, thank you.

Committed. Thank you all.

Automatically closed - issue fixed for 2 weeks with no activity.