Skip to content

Conversation

Callisto13
Copy link
Member

@Callisto13 Callisto13 commented Nov 7, 2022

Previous bins report the incorrect version: (eg after downloading
0.2.0 the binary tells me it is v0.1.2-13-gc533a0e).

I am not sure why this happens, perhaps the action cannot see the new
tag? This doesn't make much sense as it is a new tag which triggers
the release. idk.

After messing around with another binary, I think this should save the
correct thing.

Closes #508

Previous bins report the incorrect version: (eg after downloading
0.2.0 the binary tells me it is `v0.1.2-13-gc533a0e`).

I am not sure why this happens, perhaps the action cannot see the new
tag? This doesn't make much sense as it is a new tag which triggers
the release. idk.

After messing around with another binary, I think this should save the
correct thing.
@Callisto13 Callisto13 changed the title wip fix: ensure correct version is set on binaries Nov 7, 2022
@Callisto13 Callisto13 added the kind/bug Something isn't working label Nov 7, 2022
@Callisto13 Callisto13 marked this pull request as ready for review November 7, 2022 14:01
@Callisto13 Callisto13 requested a review from yitsushi November 7, 2022 14:01
@codecov-commenter
Copy link

Codecov Report

Base: 56.57% // Head: 56.53% // Decreases project coverage by -0.03% ⚠️

Coverage data is based on head (4ba5fd1) compared to base (bead002).
Patch has no changes to coverable lines.

❗ Current head 4ba5fd1 differs from pull request most recent head 1b7aceb. Consider uploading reports for the commit 1b7aceb to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #584      +/-   ##
==========================================
- Coverage   56.57%   56.53%   -0.04%     
==========================================
  Files          57       57              
  Lines        2768     2768              
==========================================
- Hits         1566     1565       -1     
  Misses       1061     1061              
- Partials      141      142       +1     
Impacted Files Coverage Δ
pkg/process/process.go 37.73% <0.00%> (-1.89%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@Callisto13 Callisto13 merged commit 4cf1378 into main Nov 8, 2022
@Callisto13 Callisto13 deleted the version branch November 8, 2022 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

flintlockd version returns confusing result
3 participants