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

fix metalist TOC. Awesome go linters #5493

Merged
merged 3 commits into from Aug 26, 2021

Conversation

davorpa
Copy link
Member

@davorpa davorpa commented Aug 24, 2021

What does this PR do?

Add resource(s) | Improve repo

For resources

Description

Add "Awesome Go linters" metalist

Why is this valuable (or not)?

How do we know it's really free?

Is a MIT repo

For book lists, is it a book? For course lists, is it a course? etc.

An Awesome list

Checklist:

  • Read our contributing guidelines
  • Search for duplicates.
  • Include author(s) and platform where appropriate.
  • Put lists in alphabetical order, correct spacing.
  • Add needed indications (PDF, access notes, under construction)

Followup

  • Check the output of Travis-CI for linter errors!

@davorpa davorpa marked this pull request as ready for review August 24, 2021 16:10
@eshellman
Copy link
Collaborator

not a book

@davorpa
Copy link
Member Author

davorpa commented Aug 24, 2021

not a book

Then it goes to... or in the other hand, other awesome lists are not valid in this repo?

@eshellman
Copy link
Collaborator

the meta-lists section is for lists of free books; there are edge cases. we have lists for courses, cheatsheets and other things but we can't cover all resources, however useful they might be.

@davorpa davorpa changed the title Add "Awesome go linters" metalist fix metalist TOC. Awesome go linters Aug 25, 2021
Other awesome resources are not handled by repo

Commit-SHA: b8302fc
Commit-URL: EbookFoundation@b8302fc
@davorpa
Copy link
Member Author

davorpa commented Aug 25, 2021

the meta-lists section is for lists of free books; there are edge cases. we have lists for courses, cheatsheets and other things but we can't cover all resources, however useful they might be.

Then, reverted

@eshellman eshellman merged commit 57aa0ec into EbookFoundation:master Aug 26, 2021
@davorpa davorpa deleted the awesome-go-linters branch October 14, 2021 01:28
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.

None yet

2 participants