toolshed/abra
toolshed/abra
4
2
Fork 18
Code Issues 27 Pull Requests 3 Packages Releases 40 Activity

fix #561 #565

Open
iexos wants to merge 1 commits from iexos/abra:main into main
pull from: iexos/abra:main
merge into: toolshed:main
Conversation 3 Commits 1 Files Changed 1 +1 -1
iexos commented 2025-05-27 15:27:26 +00:00
First-time contributor
Copy Link

this makes updates show up again for me

this makes updates show up again for me
❤️ 1 🎉 1
iexos added 1 commit 2025-05-27 15:27:26 +00:00
fix: app ls -S didn't show updates sometimes (#561)
All checks were successful
continuous-integration/drone/pr Build is passing
Details
7a99e0485b
decentral1se commented 2025-05-28 07:09:29 +00:00
Owner
Copy Link

Oh dear, it's more fallout from #549 😱 Thanks @iexos! Do you have time to write some tests for this? It seems this code path is pretty key and there is a big chance for regressions, as I have caused this bug! We somehow need to confirm that updates are being shown in the output. I can try when I get time next but not sure when that will be.

Oh dear, it's more fallout from https://git.coopcloud.tech/toolshed/abra/pulls/549 😱 Thanks @iexos! Do you have time to write some tests for this? It seems this code path is pretty key and there is a big chance for regressions, as I have caused this bug! We somehow need to confirm that updates are being shown in the output. I can try when I get time next but not sure when that will be.
iexos commented 2025-06-03 10:32:17 +00:00
Author
First-time contributor
Copy Link

I don't know if I find the space to write the tests anytime soon. Also because this is my first golang coding past a hello world...

I don't know if I find the space to write the tests anytime soon. Also because this is my first golang coding past a hello world...
decentral1se commented 2025-06-03 15:11:30 +00:00
Owner
Copy Link

@iexos no worries, I can pick it up from here! If you want to hack more on abra, you know where to find me 😌

Thanks for the fix.

@iexos no worries, I can pick it up from here! If you want to hack more on `abra`, you know where to find me 😌 Thanks for the fix.
All checks were successful
continuous-integration/drone/pr Build is passing
Details
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u main:iexos-main
git checkout iexos-main
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something is not working
build
go build related issues
ci/cd
Building things with CI/CD
contributing
Dealing with contributing
critical fix
https://docs.coopcloud.tech/federation/resolutions/passed/010/
design
UI/UX
documentation
Documenting all the things
duplicate
This issue or pull request already exists
enhancement
New feature
help wanted
Need some help
invalid
Something is wrong
meta
relating to the project or repo itself
question
More information is needed
release
Release management
release-candidate
Related to the new release candidate
security
Security related
test
Unit/integration testing
wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
3wordchant abra-bot ammaratef45 Apfelwurm carla cas decentral1se fauno kawaiipunk knoflook lambdabundesverband moritz p4u1 simon trav val yksflip
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: toolshed/abra#565
No description provided.
Delete Branch "iexos/abra:main"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?