drupol / phpcsfixer-configs-php

Base PHP CS Fixer configurations.
MIT License
5 stars 2 forks source link

New release please #73

Open FlorentTorregrosa opened 3 weeks ago

FlorentTorregrosa commented 3 weeks ago

Hi,

Would it be possible to tag new releases for this repository and for https://github.com/drupol/phpcsfixer-configs-drupal please?

github-actions[bot] commented 2 weeks ago

Since this issue has not had any activity within the last 5 days, I have marked it as stale. I will close it if no further activity occurs within the next 5 days.

FlorentTorregrosa commented 2 weeks ago

No, please bot, keep it opened.

github-actions[bot] commented 1 week ago

Since this issue has not had any activity within the last 5 days, I have marked it as stale. I will close it if no further activity occurs within the next 5 days.

FlorentTorregrosa commented 1 week ago

@drupol , Hi,

I would like to open a discussion. Maybe also with @WengerK?

You are very reactive on PR, thank you :pray:.

But from what I remember, you told me that you are no more using https://github.com/drupol/phpcsfixer-configs-drupal and https://github.com/drupol/phpcsfixer-configs-php, and I don't want to bother you making new releases or other maintenance stuff if you are not using it anymore.

So I would like to ask you if you would be ok if I open a repository on Drupal.org that would become the official phpcsfixer configuration for Drupal, or maybe it could be integrated in Coder?

If you are ok with this proposal, I will then need to discuss with Coder maintainers and or Gitlab CI on drupal.org team to know if it would interest them. But I wanted to ask you first of course.

Thanks for any reply :)

drupol commented 1 week ago

Hello Florent,

As this project is open-source, I have no objections to it being forked or further improved. Feel free to proceed if you believe that’s the best approach. Although I no longer use Drupal, moving the project to a more appropriate environment might be a good idea.

I do have only 2 requests:

WengerK commented 1 week ago

For my point of view, I'm not sure if a none-module should be on drupal.org. What is the recommendation from Drupal.org team ?

See drush being on GitHub Nop ?

FlorentTorregrosa commented 1 week ago

Hi,

Thanks both of you for your feedbacks.

On https://www.drupal.org/node/add there is the "General project", which results in a package automatically published on packagist.

Drush is on Github, there is also Coder. So it is not mandatory.

My main point is to not have @drupol bothered if he does not have the time.

And second point, discuss with Coder and/or drupal.org team what is the best to promote the work done here to become something official if possible and avoid potential duplicated work.

I will reached them so we can continue the discussion.

And without hurry :)

WengerK commented 1 week ago

Thanks for your answer, also in order to have a better reactivity, we could discuss to pass you as maintainer (alongside with me) for this project and also create a Channel on the Drupal Slack (to ease our collaboration but also to be transparent with the Drupal community and how to reach us.

For my point of view, many project outside of modules are on Github and I would prefer keeping this repo on Github:

Waiting on your answers from the Drupal Association Team

One last thing, you requests a release for for drupol/phpcsfixer-configs-drupal but I don't see any changes since last tag on https://github.com/drupol/phpcsfixer-configs-drupal/blob/master/CHANGELOG.md

and the requirement on drupol/phpcsfixer-configs-php is already permissive enought "drupol/phpcsfixer-configs-php": "^2.0" - not sure we need to make a tag for drupol/phpcsfixer-configs-drupal- nop ?

FlorentTorregrosa commented 1 week ago

For drupol/phpcsfixer-configs-drupal, because the 2 last commits did not update CHANGELOG.md: https://github.com/drupol/phpcsfixer-configs-drupal/commits/master/

Hum, I don't remember having problem with the requirements on the drupal config repository.

FlorentTorregrosa commented 1 week ago

So, I asked one of the Coder maintainer (I forgot to ask permission to mention him).

As in Coder, there is the tool to also fix the CS (since 4/5 years), there is no need to have a second tool to handle CS alongside Coder.

So I guess, we can keep the two repositories like they are right now. Just keeping it alive and maintained.

And if you are ok to give me the access so I can make merge and releases, I would be able to do it myself.

In fact, when it is just configuration adjustments, I am not blocked because I already have my overrides in https://gitlab.com/florenttorregrosa-drupal/docker-drupal-project/-/tree/11.x/scripts/quality/phpcs_fixer?ref_type=heads, it is just that I want my overrides to be only for my personal preferences not official changes :). I am blocked only when new code is needed for new rules, which happens not very often :)

drupol commented 1 week ago

I just invited you.

FlorentTorregrosa commented 1 week ago

Thanks, invitation accepted.

FlorentTorregrosa commented 5 days ago

Currently removing Scrutinizer in https://github.com/drupol/phpcsfixer-configs-php/pull/74 as it is broken and also removed in the drupal repository.

But it seems that a task is still expected and I think I don't have permissions to access CI config for this repository.

drupol commented 4 days ago

Can we close this issue now ?

FlorentTorregrosa commented 3 days ago

Hi, I will try to get the time to make a new release soon. I will close the issue then.