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 problems with latest release #146

Merged
merged 8 commits into from
Aug 23, 2024
Merged

Conversation

afinetooth
Copy link
Member

@afinetooth afinetooth commented Aug 22, 2024

⚡ Summary

Fix problems with last attempted release:

☑️ Checklist

  • Fix syntax error in src/coverage_reporter.cr
  • Make sure all tar zipped archives unzip a binary named coveralls
  • Fix syntax error in workflow step, "Create GitHub release"
  • Update README for new, dual architecture Linux binaries

… binary. Fix syntax error in . Force homebrew formula update to avoid issues with existing PRs.
Copy link

coveralls-official bot commented Aug 22, 2024

Pull Request Test Coverage Report for Build 10514373630

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 93.983%

Totals Coverage Status
Change from base Build 10493846803: 0.0%
Covered Lines: 906
Relevant Lines: 964

💛 - Coveralls

…ldx is not dropping the binaries where expected
… and build steps, and leverage the fact that buildx automatically creates architecture-specific subdirectories for each asset it builds.
…ns in the crystal build steps in Dockerfile.
@afinetooth afinetooth merged commit 41e9bd4 into master Aug 23, 2024
10 of 11 checks passed
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.

1 participant