Skip to content
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

flymake-phpcs has been archived #83

Open
tarsius opened this issue Nov 13, 2020 · 5 comments
Open

flymake-phpcs has been archived #83

tarsius opened this issue Nov 13, 2020 · 5 comments
Assignees

Comments

@tarsius
Copy link

tarsius commented Nov 13, 2020

@senda-akiha has archive flymake-phpcs. You probably should stop requiring it in drupal-mode.

Also see melpa/melpa#7225.

@issue-label-bot
Copy link

Issue Label Bot is not confident enough to auto-label this issue. See dashboard for more details.

@arnested
Copy link
Owner

Thank you, @tarsius.

I will look into this (soon hopefully).

@tarsius
Copy link
Author

tarsius commented Feb 4, 2021

Friendly ping 🏓

@arnested
Copy link
Owner

arnested commented Feb 9, 2021

Hi @tarsius

Thank you for the friendly ping. I actually forgot this issue.

drupal-mode doens't really require flymake-phpcs

It's true that we actually make a require in drupal/flymake-phpcs.el but that file is only loaded after flymake-phpcs is loaded, see drupal-mode.el line 899.

The way drupal-mode supports flymake-phpcs is that if a user has configured flymake-phpcs themselves then we load the file and adds support for it in drupal-mode.

I think this should be safe even now that flymale-phpcs is archived.

But we should probably mention in the documentation that it is archived and that we actually support flycheck.

@arnested arnested self-assigned this Feb 9, 2021
@tarsius
Copy link
Author

tarsius commented Feb 9, 2021

Best outcome would be if someone(tm) volunteered to maintain flymake-phpcs. Ideally someone who prefers flymake over flycheck. But the package is trivial enough for someone who is at least familiar with it to maintain it. Can't be me since I use neither fly* nor php.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants