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

refactor: remark-lint:awesome-github #182

Closed
wants to merge 3 commits into from
Closed

Conversation

tapyu
Copy link

@tapyu tapyu commented Dec 24, 2023

Closes #180

I accidentally deleted the repository used to create the PR #181 . That is why I opened a new one with the same proposals :)

@@ -65,7 +65,7 @@ const githubRule = lintRule('remark-lint:awesome-github', async (ast, file) => {
file.message('License was not detected by GitHub');
}
} catch {
file.message('Awesome list must reside in a valid git repository');
file.message('.git/ directory not found');
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I propose to change "Awesome list must reside in a valid git repository" to ".git/ directory not found'" in order to make it clearer.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this is better.

@@ -20,7 +20,7 @@ Includes a bunch of [general Markdown rules](config.js) and some [Awesome specif

The CLI requires Node.js and Git.

Type the command `npx awesome-lint` followed by the URL of the repo you want to check:
In a git directory, type the command `npx awesome-lint` followed by the URL of the repo you want to check:
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This becomes moot with #180 (comment)

@sindresorhus
Copy link
Owner

I think it's better to properly fix the issue: #180 (comment)

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