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

Go compilers category add go2hx #5631

Merged
merged 5 commits into from
Mar 3, 2025
Merged

Conversation

PXshadow
Copy link
Contributor

@PXshadow PXshadow commented Mar 1, 2025

We want to ensure high quality of the packages. Make sure that you've checked the boxes below before sending a pull request.

  • I have read the Contribution Guidelines

  • I have read the Maintainers Note

  • I have read the Quality Standards

  • The repo documentation has a pkg.go.dev link.

  • The repo documentation has a coverage service link.

  • The repo documentation has a goreportcard link.

  • The repo has a version-numbered release and a go.mod file.

  • The repo has a continuous integration process that automatically runs tests that must pass before new pull requests are merged.

  • Continuous integration is used to attempt to catch issues prior to releasing this package to end-users.

This project does not have a working pkg.go.dev, because the Go part of the compiler is really only used to perform reflection on the typed AST and send it to the Haxe side via JSON.

No coverage service is linked because the test coverage comes from running the Go stdlib in test mode and related unit tests from Go and Yaegi test suites using a regression system to make sure no passing test is able to regress in the future.

Please provide some links to your package to ease the review

Pull Request content

  • The package has been added to the list in alphabetical order.
  • The package has an appropriate description with correct grammar.
  • As far as I know, the package has not been listed here before.

Category quality

Please delete one of the following lines:

  • The packages around my addition still meet the Quality Standards.

  • esp32-transpiler: because the last commit was 2 years ago, and no activity on the last made issues for more then 2 years.

  • c2go archived and unfinished.

  • tardisgo On ice, last commit 9 years ago.

Although the project is written overwhelming in Haxe code, I believe this project fulfills "be generally useful to the wider community of Go programmers;".

The stdgo folder is the Go stdlib compiled into Haxe code per passing commit using Github actions. This is where the huge code size comes from, and solutions are being looked into, to have the compiler compile only the needed parts each time.

Disclaimer: I am the primary dev working on the compiler submitted, and Elliott who created tardisgo is actively working on the project in question as well.

Copy link

github-actions bot commented Mar 1, 2025

Thank you for contributing to awesome-go. We will review your contribution as soon as possible.

Make sure you add the links in the body of the pull request that are requested in the contribution guide:

  • repo link
  • pkg.go.dev
  • goreportcard.com
  • coverage

Your project is under review. It may take a few days to be approved.

@phanirithvij phanirithvij merged commit 86358b2 into avelino:main Mar 3, 2025
2 checks passed
@phanirithvij
Copy link
Collaborator

added two of those back, there is some activity and utility still. can remove those at a later time.

@PXshadow PXshadow deleted the patch-1 branch March 3, 2025 06:44
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.

2 participants