Problem/Motivation

Since Jun 2023, Drupal.org enabled GitLab CI for testing purposes. GitLab CI is now preferred over DrupalCI for testing.
This issue is to track progress towards getting a .gitlab-ci.yml file so that we can migrate away from the deprecated DrupalCI to GitLab CI.

GitLab CI documentation: https://www.drupal.org/docs/develop/git/using-gitlab-to-contribute-to-dr...

Benefits:

1) Testing is about five times faster
@see: https://www.drupal.org/about/core/blog/drupal-cores-gitlab-ci-testing-is...

2) Ability to configure tests in a more obvious and flexible way.

3) Great UI!

4) While we don't have tests in the module, we would still benefit from automated code style checks.

API changes

No API changes.

This task is devoted to the LutskGCW24, please, don't work with it until it starts.

Issue fork entityreference_live_preview-3417254

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

snap_x created an issue. See original summary.

oleksandr p.’s picture

oleksandr p. commented
Assigned: Unassigned » oleksandr p.
oleksandr p.’s picture

oleksandr p. commented
Assigned: oleksandr p. » Unassigned
Status: Active » Needs review

I added the default Gitlab CI template for the project. Created a new MR.

Oleksandr P. opened merge request !1

_shy’s picture

_shy
Primary language Ukrainian
Location Ukraine, Lutsk 🇺🇦
commented
Status: Needs review » Reviewed & tested by the community

Looks fine. Added to merge queue.

_shy’s picture

_shy
Primary language Ukrainian
Location Ukraine, Lutsk 🇺🇦
commented
Status: Reviewed & tested by the community » Needs work

Just found that MR created in the wrong branch. Please, revert these changes from the 7.x branch and create a new one to the 8.x.

ViacheslavMiakota’s picture

ViacheslavMiakota commented
Assigned: Unassigned » ViacheslavMiakota

ViacheslavMiakota opened merge request !2

ViacheslavMiakota’s picture

ViacheslavMiakota commented
Assigned: ViacheslavMiakota » Unassigned
Status: Needs work » Needs review
snap_x’s picture

snap_x
Location 🇺🇦 Ukraine, Lutsk
commented
Status: Needs review » Needs work

Add "SKIP_PHPUNIT: 1" variable please
Check an example here

ViacheslavMiakota’s picture

ViacheslavMiakota commented
Assigned: Unassigned » ViacheslavMiakota
ViacheslavMiakota’s picture

ViacheslavMiakota commented
Assigned: ViacheslavMiakota » Unassigned
Status: Needs work » Needs review

_shy opened merge request !9

_shy’s picture

_shy
Primary language Ukrainian
Location Ukraine, Lutsk 🇺🇦
commented
Issue tags: -LutskGCW24 +LutskGCW26
_shy’s picture

_shy
Primary language Ukrainian
Location Ukraine, Lutsk 🇺🇦
commented
Status: Needs review » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

  • _shy committed 80be3039 on 8.x-1.x 
    Resolve #3417254 "Set up gitlab"

_shy opened merge request !10

  • _shy committed 2fccfe4f on 8.x-1.x 
    Resolve #3417254 "Set up gitlab"

Status: Fixed » Closed (fixed)

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