Skip to content

Conversation

@YoshiRulz
Copy link

@YoshiRulz YoshiRulz commented Aug 12, 2025

Aiming to implement #69. As I said then, I'm not a Crystal programmer, so maybe Copilot can finish this.

@YoshiRulz

This comment was marked as resolved.

@YoshiRulz YoshiRulz marked this pull request as ready for review August 12, 2025 14:58
@oprypin
Copy link
Owner

oprypin commented Aug 14, 2025

I re-ran the tests. The tests fail because they have a baked-in fake copy of an API response, which is minimal and doesn't have head_sha even if it is a real attribute returned from json (should be double-checked). The tests will need to be updated.

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