Problem/Motivation

The project contains a Travis file, but this file contains outdated configuration and can be removed. Automated tests are performed on Drupal.org infrastructure since #3388682: Move automated testing to GitLab

Proposed resolution

Remove .travis.yml

Remaining tasks

  1. Write a merge request
  2. Review
  3. Commit

User interface changes

None.

API changes

None.

Data model changes

None.

Issue fork domain-3553358

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

idebr created an issue. See original summary.

idebr opened merge request !253

idebr’s picture

idebr commented
Status: Active » Needs review

The merge request implements the following changes:

  1. .travis.yml is removed
  2. Obsolete define_subdomains.sh script for drupal CI is removed
  3. References to these files are removed
mably’s picture

mably commented
Status: Needs review » Needs work

Hi @idebr, thanks for your MR.

Looks like a PHPCS warning needs to be fixed.

idebr’s picture

idebr commented
Status: Needs work » Needs review

Strictly speaking the phpcs violation is out of scope, since it fails due to a recent change in Drupal Core, see #3486996: Fix extra periods at the end of comment lines

However, since the change is trivial enough, I'll include it here

mably’s picture

mably commented

Let's get rid of Travis in 3.x 👍

  • mably committed 3e77b25d on 3.x authored by idebr 
    Issue #3553358 by idebr: Remove obsolete .travis.yml
mably’s picture

mably 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.

Status: Fixed » Closed (fixed)

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