mirror of
https://github.com/sindresorhus/awesome.git
synced 2026-07-18 04:23:47 -05:00
[PR #1394] [CLOSED] Add awesome-lint and CI to lint new awesome lists #17189
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/sindresorhus/awesome/pull/1394
Author: @transitive-bullshit
Created: 9/11/2018
Status: ❌ Closed
Base:
master← Head:feature/awesome-lint📝 Commits (5)
dfa9e0dAdd awesome-lint to test new awesome lists2babd4aAdd travis-ci1c50297Update .travis.yml2c64c54Update package.json6ee57bbUpdate lists.js📊 Changes
5 files changed (+542 additions, -0 deletions)
View changed files
➕
.gitignore(+2 -0)➕
.travis.yml(+3 -0)➕
package.json(+27 -0)➕
test/_whitelist.js(+438 -0)➕
test/lists.js(+72 -0)📄 Description
This PR incorporates awesome-lint and is aimed at providing automated linting to ease the load of reviewing new PRs.
It also adds a
.travis.ymlfile to enable travis-ci testing for PRs.Note that it is not our intention to fully automate PR reviews, as content is still very important and will always require a manual pass. Instead, we're trying to reduce the amount of time it takes to manually review and maintain the Awesome project.
For new and existing Awesome list authors, lists are not required to pass
awesome-lintyet, but it is a good idea to run the linter on your list before submitting, as it will help you find a lot of common issues and will hopefully save us all a great deal of time in the long run.For more background, see https://github.com/sindresorhus/awesome-lint/issues/39
cc @sindresorhus @itaisteinherz
Fixes #1365.
Fixes #569.
Helps with #1363.
(Edit) Note that this PR is dependent on https://github.com/sindresorhus/awesome-lint/pull/43
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.