- Issue created by @alexpott
- πͺπΈSpain fjgarlin
This just went in last week π PHPCS error in contributed module caused by core recipe.README.txt Active , and this β¨ Preserve original composer.json in the modules directory Fixed was done a while ago.
So I'm not sure this is needed anymore.
- π¬π§United Kingdom alexpott πͺπΊπ
@fjgarlin I disagree. The new code is still very fragile. For example, if the project contains a recipes folder then we're still going to place a readme (or overwrite it). We inherited this approach from Drupalspoons and whilst it has been great to get everything working it has shortcomings and breaks a principle - we should minimise (hopefully to zero) the things that we alter in code under test. The whole backup approach in β¨ Preserve original composer.json in the modules directory Fixed is complex and fraught.
- πͺπΈSpain fjgarlin
Agree that it is still far from ideal, that's why I also created this a while back #3425971: Explore the posibility of a different setup for the project, composer expansion and symlinking process β , but I haven't had the time to work on it.
I definitely agree with the approach of having two folders, one for the full project, and one for the module, from there we can expand and work things out, but my main worry is how many pipelines we fix vs how many we break, especially if they have customisations on the jobs.
In any case, this is something that we want/need to improve, so I'll leave this open as it is more recent. I'm almost sure that we would not be able to keep things BC, so if that'd be the case, we'd need to work on a strategy for it, but that's out of the scope for this issue.
I suggest creating a brand new file in the
includes
folder, which we can start as a copy ofinclude.drupalci.main.yml
and then start changing code on that new file. We'd be able to easily test it by changing a reference to the new file if/when needed. - π¬π§United Kingdom alexpott πͺπΊπ
@fjgarlin yep BC is going to be a PITA. And I think I agree we should have a way to coordinate a switch over. Another way of thinking about this might be to change this project's release strategy. This change should be a new major version and people should be able to stick on the last major version if they need and then update when they have time. Most projects will not have customised there pipelines to the point they rely on the layout so they will be fine. The projects that break can change to include the previous major version and add a ticket to update to the new major version.
- πͺπΈSpain fjgarlin
Any major effort into refactoring things should probably be done via GitLab components β¨ See we if can and/or should transfer the templates into GitLab CI/CD components Active .
I've tagged issues from the queue which I think were part of a possible 2.x conversation, see them here: https://www.drupal.org/project/issues/search?text=&projects=GitLab+Templ... β
I don't think that we have the capacity right now at the DA to work on this, as there are other priorities way ahead of this, and this project itself, while not perfect, can be considered stable and is working mostly well. So, there are no immediate plans to start working on a 2.x effort, but we can start seeing what could be needed via the previous tag at least.