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

remove cookiestxt #4364

Closed
wants to merge 1 commit into from
Closed

remove cookiestxt #4364

wants to merge 1 commit into from

Conversation

switchupcb
Copy link
Contributor

cookiestxt is scheduled to be removed from awesome-go on August 20, 2022. If you would like it to remain, fix your repository such that it passes tests with a current version of Go and shows code coverage of 80% or more. In addition, your project is failing to meet the following criteria:

  • Official releases should be at least once a year if the project is ongoing. The last release for cookiestxt was on Mar 08, 2021.
  • The project does not support generics issued in go 1.18. Projects must be compatible with Go versions issued in the last year. See https://go.dev/doc/devel/release for release history.

Once completed, post here to confirm.

Owner: @mengzhuo
Submission: #1616
Related: https://github.com/avelino/awesome-go/pulls?q=is%3Apr+cookiestxt+is%3Aclosed
Prompt: #4352
NOTE: 154 Lines of Go Code

@avelino
Copy link
Owner

avelino commented Aug 6, 2022

Thank you for contributing with awesome-go, we will revise your contribution as soon as possible.

Automation body links content check:

  • godoc.org or pkg.go.dev: True
  • goreportcard.com: False
  • coverage: False

your project is about to be approved, it's under revision, it may take a few days

@mengzhuo
Copy link
Contributor

mengzhuo commented Aug 8, 2022

I'll take a look.

@mengzhuo
Copy link
Contributor

mengzhuo commented Aug 8, 2022

I've updated coverage and reportcard, please close this PR, thanks for inform.

@switchupcb switchupcb deleted the cookiestxt branch December 5, 2022 16:55
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

4 participants