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

Resolve commit ref correctly. #212

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

blkt
Copy link
Contributor

@blkt blkt commented Nov 15, 2024

The command frizbee actions <action>@<tag> incorrectly retrieves the tag's ref rather than the commit's one. The tag's ref should instead be used to retrieve the details of the tag, which contain the ref of the commit.

This change changes getCheckSumForTag to lookup the right field using two subsequent call to GitHub.

Fixes #206

@blkt blkt self-assigned this Nov 15, 2024
@blkt blkt force-pushed the fix/frizbee-action-invalid-commit-ref branch from 1e136a0 to 22b52e1 Compare November 15, 2024 21:34
@coveralls
Copy link

coveralls commented Nov 15, 2024

Coverage Status

coverage: 52.926% (+0.09%) from 52.832%
when pulling aae8b7a on fix/frizbee-action-invalid-commit-ref
into c2a623a on main.

@blkt blkt force-pushed the fix/frizbee-action-invalid-commit-ref branch from 22b52e1 to c5a341c Compare November 15, 2024 21:37
The command `frizbee actions <action>@<tag>` incorrectly retrieves the
tag's ref rather than the commit's one. The tag's ref should instead
be used to retrieve the details of the tag, which contain the ref of
the commit.

This change changes `getCheckSumForTag` to lookup the right field
using two subsequent call to GitHub.

Fixes #206
@blkt blkt force-pushed the fix/frizbee-action-invalid-commit-ref branch from c5a341c to aae8b7a Compare November 15, 2024 21:52
@blkt blkt changed the title Retrieve commit ref correctly. Resolve commit ref correctly. Nov 15, 2024
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.

frizbee actions generates wrong/invalid commit ref?
2 participants