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
Disable failing test in PR runs for now #29242
Conversation
🤖 Thank you for contributing to CiviCRM! ❤️ We will need to test and review this PR. 👷 Introduction for new contributors...
Quick links for reviewers...
|
@eileenmcnaughton @totten I think we can completely remove this test, since |
It wouldn't be missed.... |
I'd agree that the files are deprecated - but in my copy of |
@totten Yep good point - I opened an issue here: https://lab.civicrm.org/dev/drupal/-/issues/193 and created a few PRs to work towards that. Aegir does not use this code for Drupal8+, but for Drupal7 it was still being used. I don't think many people use it though, except .. CiviCRM Spark and a handful of Symbiotic clients. |
This was backported to 5.69, so it may show up in those release-notes. Just noting that I think the test-failure points to a data problem. I've split off separate issue about that (https://lab.civicrm.org/dev/translation/-/issues/84). For |
Overview
All tests are failing on this today - let's suppress it for now - we can unsuppress once resolved but this test is really gonna rely on @totten or @mlutfy to resolve