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

Replace from Code Climate to Super-Linter #18587

Merged
merged 1 commit into from
Jun 1, 2022
Merged

Replace from Code Climate to Super-Linter #18587

merged 1 commit into from
Jun 1, 2022

Conversation

ykzts
Copy link
Member

@ykzts ykzts commented Jun 1, 2022

Since Code Climate runs in a sandbox, it is incompatible with Linter (e.g. ESLint), which relies on external libraries. It also does not support stylelint, an alternative to sass-lint, which is no longer maintained.

In this Pull Request, we are replacing Code Climate with Super-Linter, which is developed by GitHub and is compatible with GitHub Pull Requests.

@Gargron Gargron merged commit 84266f8 into main Jun 1, 2022
@Gargron Gargron deleted the super-linter branch June 1, 2022 17:22
@ClearlyClaire
Copy link
Contributor

Trying to merge the changes in glitch-soc, I am confused at it complaining on lines that haven't been changed from upstream. Also, its output is a lot less friendly than CodeClimate :/
https://github.com/glitch-soc/mastodon/runs/6704509941?check_suite_focus=true

@ykzts ykzts mentioned this pull request Jun 3, 2022
kadoshita pushed a commit to kadoshita/mastodon that referenced this pull request Nov 19, 2022
untitaker added a commit to untitaker/mastodon that referenced this pull request Jan 9, 2023
Switching to super-linter reveals too many linting errors in files I
touch in PRs.

It seems that the PR that did this: mastodon#18587 -- is largely frontend-motivated, and that backend was better off with codeclimate.

Since both codeclimate and superlinter is configured in principle
anyway, we might as well have codeclimate for the backend and
superlinter for the frontend.
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

Successfully merging this pull request may close these issues.

3 participants