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

Issues reported by awesome-lint #107

Closed
mgrojo opened this issue Feb 25, 2023 · 5 comments
Closed

Issues reported by awesome-lint #107

mgrojo opened this issue Feb 25, 2023 · 5 comments
Assignees

Comments

@mgrojo
Copy link
Collaborator

mgrojo commented Feb 25, 2023

The awesome-lint program has run for the first time; I wasn't aware of those issues. It seems the link checker was aborting the action.

https://github.com/ohenley/awesome-ada/actions/runs/4267389929/jobs/7428942700

There are some legitimate and important issues to be fixed:

  • Double links in the Contents (only first one will work).
  • Double links due to incorrect copy-paste for a book.

Other puristic ones, but easy to solve.

A few looks like there's some bug in awesome-lint.

And finally, other ones that I'm not sure if they should be solved: either because they would affect presentation or because we would lose something, like a second link to the same URL. It doesn't like either the out-of-place links that I precisely included to centralize the URL, so I don't see a solution for those.

I'll try to clean the important and easy ones, and we should discuss the remaining: whether there's a way to fix them, the rule can be disabled or the lint has to be removed altogether.

mgrojo added a commit that referenced this issue Feb 25, 2023
These issues go in the categories:
- Important ones to be solved.
- Puristic ones easy to solve.
See issue #107
mgrojo added a commit that referenced this issue Feb 25, 2023
mgrojo added a commit that referenced this issue Feb 25, 2023
Left only in the Embedded section, since it of most interest in that area than as a general sound library.

```
README.md:592:3:   https://github.com/raph-amiard/ada-synth-lib                 remark-lint:double-link
README.md:653:3:   https://github.com/raph-amiard/ada-synth-lib                 remark-lint:double-link
```
See issue #107
@mgrojo
Copy link
Collaborator Author

mgrojo commented Feb 25, 2023

I directly fixed the non-controversial issues and have made pull requests with the ones requiring review.

These are the only remaining issues if everything is merged:

README.md:1:1:   The repository should have "awesome-list" as a GitHub topic  remark-lint:awesome-github

This can be solved adding the corresponding tag, that can only be done by the owner.

README.md:194:3:   https://matrix.to/#/#ada-lang_Lobby:gitter.im                remark-lint:double-link
README.md:202:112: https://matrix.to/#/#ada-lang:matrix.org                     remark-lint:double-link

These seem like a bug in how the linter interprets URLs. See Scrum/remark-lint-double-link#10

mgrojo added a commit that referenced this issue Feb 25, 2023
Fixes these occurrences:
```
  ✖  266:122  https://github.com/AdaCore/ada_language_server               remark-lint:double-link
  ✖  289:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  302:69   List item description must start with valid casing           remark-lint:awesome-list-item
  ✖  322:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  343:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  346:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  350:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  384:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  402:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  449:3    https://github.com/AdaCore/ada_language_server               remark-lint:double-link
  ✖  492:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  523:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  537:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  562:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  573:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  574:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  590:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  695:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  724:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  725:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  735:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  736:3    Invalid list item link                                       remark-lint:awesome-list-item
```

See issue #107
@mgrojo mgrojo self-assigned this Mar 3, 2023
mgrojo added a commit to mgrojo/awesome-ada that referenced this issue Mar 4, 2023
Fixes these occurrences:
```
  ✖  266:122  https://github.com/AdaCore/ada_language_server               remark-lint:double-link
  ✖  289:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  302:69   List item description must start with valid casing           remark-lint:awesome-list-item
  ✖  322:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  343:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  346:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  350:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  384:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  402:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  449:3    https://github.com/AdaCore/ada_language_server               remark-lint:double-link
  ✖  492:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  523:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  537:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  562:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  573:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  574:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  590:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  695:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  724:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  725:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  735:3    Invalid list item link                                       remark-lint:awesome-list-item
  ✖  736:3    Invalid list item link                                       remark-lint:awesome-list-item
```

See issue ohenley#107
@mgrojo mgrojo closed this as completed in ec0ef84 Apr 7, 2023
@mgrojo
Copy link
Collaborator Author

mgrojo commented Apr 7, 2023

There's still a pending issue reported by the linter:

README.md:1:1
✖ 1:1 The repository should have "awesome-list" as a GitHub topic remark-lint:awesome-github

@ohenley I think this can only be fixed by you.

@mgrojo mgrojo reopened this Apr 7, 2023
@mgrojo
Copy link
Collaborator Author

mgrojo commented Jun 26, 2023

@ohenley hace you missed this? It should only take a minute.

@ohenley
Copy link
Owner

ohenley commented Jun 27, 2023

Yep missed it. Did add "awesome-list" as a GitHub topic.

@mgrojo
Copy link
Collaborator Author

mgrojo commented Jul 9, 2023

Great! Everything is passing now, except a timeout in https://www.ada-europe.org/ which I hope is a transitory problem. Closing this, thanks.

@mgrojo mgrojo closed this as completed Jul 9, 2023
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