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.

Sign up for GitHub

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

Jump to bottom

Chore: Tidy Badges #62

Closed
3 tasks done
nelsonic opened this issue Jun 7, 2023 · 6 comments · Fixed by #63
Closed
3 tasks done

Chore: Tidy Badges #62

nelsonic opened this issue Jun 7, 2023 · 6 comments · Fixed by #63
Assignees
@LuchoTurtle
Labels
chore a tedious but necessary task often paying technical debt priority-1 Highest priority issue. This is costing us money every minute that passes. T25m Time Estimate 25 Minutes tech-debt A feature/requirement implemented in a sub-optimal way & must be re-written technical A technical issue that requires understanding of the code, infrastructure or dependencies

Comments

@nelsonic
Copy link
Member

nelsonic commented Jun 7, 2023 •
edited

At present the badges in this repo README.md has two badges that are "broken":
https://www.npmjs.com/package/decache
image

Todo

@LuchoTurtle once these two micro tasks are complete,
please take a quick look at all the open issues and see what can be done about them. 🙏
@nelsonic nelsonic added chore a tedious but necessary task often paying technical debt technical A technical issue that requires understanding of the code, infrastructure or dependencies tech-debt A feature/requirement implemented in a sub-optimal way & must be re-written T5m Quick tasks that take 5 mins or less. See: GTD 2 min rule. labels Jun 7, 2023
@nelsonic nelsonic added T25m Time Estimate 25 Minutes priority-1 Highest priority issue. This is costing us money every minute that passes. and removed T5m Quick tasks that take 5 mins or less. See: GTD 2 min rule. labels Jun 12, 2023
@LuchoTurtle
Copy link
Member

@nelsonic I have the build badge working but the code analysis doesn't seem to have any snapshots.

https://codeclimate.com/github/dwyl/decache/builds

image

I've tried to refresh but nothing happens. Do I need access?

LuchoTurtle added a commit that referenced this issue Jun 12, 2023
@LuchoTurtle
fix: Fixing build badge. #62
a8a9a6e
@LuchoTurtle LuchoTurtle mentioned this issue Jun 12, 2023
Merged
@nelsonic nelsonic reopened this Jun 12, 2023
@LuchoTurtle
Copy link
Member

This issue is still assigned to me, I assume because of code analysis.

@nelsonic I have the build badge working but the code analysis doesn't seem to have any snapshots.

codeclimate.com/github/dwyl/decache/builds

Have you got any updates from this, @nelsonic ?

@nelsonic
Copy link
Member Author

Yeah, this doesn't appear to be working ...

image

I've tried triggering the refresh several times and it does not update. ♻️

image

Let's just disable (comment out) the button for now. ✂️

nelsonic added a commit that referenced this issue Jun 21, 2023
@nelsonic
Chore: remove code climate badge as not working see #62
c4e834c
@nelsonic nelsonic mentioned this issue Jun 21, 2023
Merged
@nelsonic
Copy link
Member Author

PR: #64

@LuchoTurtle
Copy link
Member

Closing as #64 has been merged.

@nelsonic nelsonic mentioned this issue Jun 21, 2023
Closed
4 tasks
@nelsonic
Copy link
Member Author

Almost ... ⏳
There's #OneMoreThing -> #65 1️⃣

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@LuchoTurtle LuchoTurtle

Labels
chore a tedious but necessary task often paying technical debt priority-1 Highest priority issue. This is costing us money every minute that passes. T25m Time Estimate 25 Minutes tech-debt A feature/requirement implemented in a sub-optimal way & must be re-written technical A technical issue that requires understanding of the code, infrastructure or dependencies
Projects
dwyl app kanban
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[PR] Fixes README badges dwyl/decache
2 participants
@nelsonic @LuchoTurtle