Skip to content

Add jseq. - #6631

Open
bobg wants to merge 2 commits into
avelino:mainfrom
bobg:bobg/jseq
Open

Add jseq.#6631
bobg wants to merge 2 commits into
avelino:mainfrom
bobg:bobg/jseq

Conversation

@bobg

@bobg bobg commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Required links

Provide the links below. Our CI will automatically validate them.

Pre-submission checklist

Repository requirements

These are validated automatically by CI:

  • The repo has a go.mod file and at least one SemVer release (vX.Y.Z).
  • The repo has an open source license.
  • The repo documentation has a pkg.go.dev link.
  • The repo documentation has a goreportcard link (grade A- or better).
  • The repo documentation has a coverage service link.

These are recommended and reported as warnings:

  • The repo has a continuous integration process (GitHub Actions, etc.).
  • CI runs tests that must pass before merging.

Pull Request content

These are validated automatically by CI:

  • This PR adds/removes/changes only one package.
  • The package has been added in alphabetical order.
  • The link text is the exact project name.
  • The description is clear, concise, non-promotional, and ends with a period.
  • The link in README.md matches the forge link above.

Category quality

Note: new categories require a minimum of 3 packages.

Packages added a long time ago might not meet the current guidelines anymore. It would be very helpful if you could check 3-5 packages above and below your submission to ensure they still meet the Quality Standards.

Please delete one of the following lines:

  • I removed the following packages around my addition: (please give a short reason for each removal)

  • I removed JSON-to-Go because https://github.com/mholt/json-to-go is not written in Go (it's Typescript) and does not contain a go.mod file.

  • Likewise JSON-to-Proto, which doesn't even have anything to do with Go.

Thanks for your PR, you're awesome! 😎

@github-actions

Copy link
Copy Markdown

Automated Quality Checks

Required checks

Repo: accessible, has go.mod and SemVer release
pkg.go.dev: OK
Go Report Card: OK (grade unknown)

Additional checks

License: MIT
Maturity: repo has 5+ months of history
CI/CD: GitHub Actions workflows detected
README: present
⚠️ Coverage: missing from PR body

How to fix: Add a coverage service link to your PR description:

Coverage: https://app.codecov.io/gh/your-org/your-project

Popular options: [Codecov](https://codecov.io), [Coveralls](https://coveralls.io). Integrate one with your CI to track coverage automatically.


_These checks are automated and do not replace maintainer review. See [CONTRIBUTING.md](https://github.com/avelino/awesome-go/blob/main/CONTRIBUTING.md) for full guidelines._

---

## PR Diff Validation

### Content checks

✅ **Files changed**: only README.md
❌ **Single item**: 1 added, 2 removed (expected exactly 1 change per PR)
  > **How to fix:** Each PR should add, remove, or change only **one** package.
  > Please split this into separate PRs — one package per PR.
✅ **Link consistency**: README link matches forge link in PR body
✅ **Link text**: matches repository name
✅ **Description**: ends with punctuation
✅ **Description tone**: no promotional language detected
✅ **Category size**: JSON has 40 items

---
> **Action needed:** one or more content checks failed. Please review the [contribution guidelines](https://github.com/avelino/awesome-go/blob/main/CONTRIBUTING.md).

_Automated diff validation — does not replace maintainer review._
<!-- Sticky Pull Request Commentpr-quality-check -->

@bobg

bobg commented Aug 29, 2026

Copy link
Copy Markdown
Contributor Author

Hm, despite what the automatic check above claims, there is in fact a coverage link in my PR description.

Also, the "single item" check overlooks the conflicting requirement that I remove entries around my addition that do not meet the quality standards (which I have done).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant