Skip to content

recentStars does not return#99

Closed
dhufe wants to merge 93 commits intomuesli:masterfrom
dhufe:master
Closed

recentStars does not return#99
dhufe wants to merge 93 commits intomuesli:masterfrom
dhufe:master

Conversation

@dhufe
Copy link

@dhufe dhufe commented Feb 26, 2025

Hey Christian,

I noticed a dead lock when I was building a new container image. The function recentStars never returns, since the loop also continues if the found stars are zero.

I also added a method for getting the lastest issues opened by the query user.

Feel free to use it, if interested.

Best regards,
Daniel

dhufe and others added 30 commits February 24, 2025 10:25
chore: Configure Renovate

See merge request pub/markscribe!1
…lop'

chore(deps): update github.com/shurcool/graphql digest to ed46e5a

See merge request pub/markscribe!3
…elop'

chore(deps): update github.com/shurcool/githubv4 digest to 4829585

See merge request pub/markscribe!2
chore(deps): update module github.com/caarlos0/env/v6 to v11

See merge request pub/markscribe!7
chore(deps): update module golang.org/x/oauth2 to v0.33.0

See merge request pub/markscribe!6
chore(deps): update module github.com/mmcdole/gofeed to v1.3.0

See merge request pub/markscribe!5
chore(deps): update alpine docker tag to v3.23

See merge request pub/markscribe!4
# Conflicts:
#	Dockerfile

diff --git a/Dockerfile b/Dockerfile
index 987136d..6f10583 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -1,9 +1,9 @@
-FROM golang:alpine3.20 AS build
+FROM docker.io/golang:1.25.5-alpine3.22 AS build
 WORKDIR /go/markscribe
 COPY . .
 RUN go build

-FROM alpine:3.23 AS prod
+FROM alpine:3.22 AS prod

 COPY --from=build /go/markscribe/markscribe /go/bin/markscribe
 ENTRYPOINT ["/go/bin/markscribe"]
\ No newline at end of file
diff --git a/go.mod b/go.mod
index 0e7a5b8..d95a795 100644
--- a/go.mod
+++ b/go.mod
@@ -1,6 +1,6 @@
 module github.com/muesli/markscribe

-go 1.25
+go 1.24.5

 require (
 	github.com/KyleBanks/goodreads v0.0.0-20200527082926-28539417959b
@dhufe dhufe closed this by deleting the head repository Dec 6, 2025
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.

1 participant