AFAICT, none of the travis configuration stuff is specific to flag. It should be something generally available to all contrib modules, rather than in our codebase.

CommentFileSizeAuthor
#5 remove-travis-2594275-5.patch5.45 KBsdstyles
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joachim created an issue. See original summary.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented

That's not how travis works. If you want it, you need that config.

However, it doesn't make sense to have in a d.o module that doesn't actually use travis. So yes, it can be removed.

socketwench’s picture

socketwench CreditAttribution: socketwench as a volunteer commented
Issue tags: +Novice

Should be a pretty easy novice patch. Remove the following:

  • .travis.yml
  • .travis/
anup.singh’s picture

anup.singh CreditAttribution: anup.singh as a volunteer and at Faichi Solutions Pvt Ltd commented
Assigned: Unassigned » anup.singh
sdstyles’s picture

sdstyles CreditAttribution: sdstyles at FFW commented
Status: Active » Needs review
FileSize
remove-travis-2594275-5.patch5.45 KB

Here is the patch

anup.singh’s picture

anup.singh CreditAttribution: anup.singh as a volunteer and at Faichi Solutions Pvt Ltd commented
Status: Needs review » Reviewed & tested by the community

Hi sdstyles,

I was about to give patch, No issues now.
Patch looks good to me.

Thanks

anup.singh’s picture

anup.singh CreditAttribution: anup.singh as a volunteer and at Faichi Solutions Pvt Ltd commented
Assigned: anup.singh » Unassigned
joachim’s picture

joachim CreditAttribution: joachim commented
Status: Reviewed & tested by the community » Fixed

Fixed. Thanks everyone!

  • joachim committed a7576f8 on 8.x-4.x authored by sdstyles 
    Issue #2594275 by sdstyles: Removed files for integration with Travis CI...

Status: Fixed » Closed (fixed)

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