~xenrox

Wuppertal

https://xenrox.net

~xenrox/hut-dev

Last active 24 days ago
View more

Recent activity

[PATCH git.sr.ht] blueprints/repo: Update deprecated pygit2 usage 7 minutes ago

From Thorben Günther to ~sircmpwn/sr.ht-dev

---
And another one ...

 gitsrht/blueprints/repo.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gitsrht/blueprints/repo.py b/gitsrht/blueprints/repo.py
index 8d5b2c3..07d19ba 100644
--- a/gitsrht/blueprints/repo.py
+++ b/gitsrht/blueprints/repo.py
@@ -433,7 +433,7 @@ def blame(owner, repo, ref, path):
                path="/".join(path)))

        try:
[message trimmed]

[PATCH git.sr.ht v2] blueprints/repo: Fix more pygit2 regressions 16 hours ago

From Thorben Günther to ~sircmpwn/sr.ht-dev

Otherwise this will happen:
  File "/usr/lib/python3.12/site-packages/gitsrht/blueprints/repo.py", line 148, in summary
    tags = [(ref, git_repo.get(git_repo.references[ref].raw_target))
                               ~~~~~~~~~~~~~~~~~~~^^^^^
  File "/usr/lib/python3.12/site-packages/pygit2/references.py", line 41, in __getitem__
    return self._repository.lookup_reference(name)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: bad argument type for built-in operation
---
v2: Add one more case

 gitsrht/blueprints/repo.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
[message trimmed]

[PATCH git.sr.ht] blueprints/repo: Fix more pygit2 regressions 17 hours ago

From Thorben Günther to ~sircmpwn/sr.ht-dev

Otherwise this will happen:
  File "/usr/lib/python3.12/site-packages/gitsrht/blueprints/repo.py", line 148, in summary
    tags = [(ref, git_repo.get(git_repo.references[ref].raw_target))
                               ~~~~~~~~~~~~~~~~~~~^^^^^
  File "/usr/lib/python3.12/site-packages/pygit2/references.py", line 41, in __getitem__
    return self._repository.lookup_reference(name)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: bad argument type for built-in operation
---
 gitsrht/blueprints/repo.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gitsrht/blueprints/repo.py b/gitsrht/blueprints/repo.py
index f1785cf..c6d3254 100644
[message trimmed]

[PATCH core.sr.ht] config: Add /etc/sr.ht/config.ini to paths 2 days ago

From Thorben Günther to ~sircmpwn/sr.ht-dev

At the moment all ".ini" files under /etc/sr.ht are loaded - even if
/etc/sr.ht/config.ini exists. To conform with the Go equivalent [1],
only config.ini will be loaded if it exists.

[1]: https://git.sr.ht/~sircmpwn/core-go/commit/3c1346e6bbc37884ca5eb390e22728e4c40e3fa5
---
 srht/config.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/srht/config.py b/srht/config.py
index 4504769..8ac0da3 100644
--- a/srht/config.py
+++ b/srht/config.py
@@ -23,8 +23,8 @@ def load_one(path):
[message trimmed]

Re: [PATCH v2] doc: Added link to manpage in README.md 24 days ago

From Thorben Günther to ~xenrox/hut-dev

Pushed, thanks!

[PATCH lists.sr.ht] api/graph: Fix list description validation 28 days ago

From Thorben Günther to ~sircmpwn/sr.ht-dev

Since description is a nullable field, it should be possible to unset
it.
---
 api/graph/schema.resolvers.go | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/api/graph/schema.resolvers.go b/api/graph/schema.resolvers.go
index 3b112d1..366049e 100644
--- a/api/graph/schema.resolvers.go
+++ b/api/graph/schema.resolvers.go
@@ -611,10 +611,12 @@ func (r *mutationResolver) UpdateMailingList(ctx context.Context, id int, input
	valid := valid.New(ctx).WithInput(input)
	query := sq.Update(`list`).PlaceholderFormat(sq.Dollar)
[message trimmed]

[PATCH todo.sr.ht] api/graph: Fix tracker description validation 30 days ago

From Thorben Günther to ~sircmpwn/sr.ht-dev

Since description is a nullable field, it should be possible to unset
it.
---
 api/graph/schema.resolvers.go | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/api/graph/schema.resolvers.go b/api/graph/schema.resolvers.go
index 751bbc7..fd8deea 100644
--- a/api/graph/schema.resolvers.go
+++ b/api/graph/schema.resolvers.go
@@ -239,12 +239,14 @@ func (r *mutationResolver) UpdateTracker(ctx context.Context, id int, input map[

	validation := valid.New(ctx).WithInput(input)
[message trimmed]

Re: [PATCH v2] all: Streamline reading of input a month ago

From Thorben Günther to ~xenrox/hut-dev

Thank you for the patch, this is a really nice behaviour improvement!

I have edited it a bit by running "gofmt", because there were some
unnecessary semicolons.

Re: [PATCH] graphql: test STDIN for terminal-ness instead of STDOUT a month ago

From Thorben Günther to ~xenrox/hut-dev

On Thu, Jun 13, 2024 at 11:36:55PM GMT, Jens Schmidt wrote:
> [Hmph, my git-sendemail-foo is still suboptimal - the following
> was intended to be sent as introduction to the patch I just
> sent but din't come through.]

It came through just fine [1], but I guess it is not displayed as
desired since sourcehut's web interface expects new patch versions in
a new thread instead of setting "--in-reply-to".

As for the patch itself, I will probably take a look Friday but I can
grok moving the "stdin" logic out of the "termfmt" package.

[1]: https://lists.sr.ht/~xenrox/hut-dev/%3C20240509155631.13645-1-farblos@vodafonemail.de%3E#%3C20240613213222.15502-1-farblos@vodafonemail.de%3E

Re: [RFC] Implementing auto-cursor support in "hut graphql" a month ago

From Thorben Günther to ~xenrox/hut-dev

On Sat, Jun 08, 2024 at 12:50:42PM GMT, Jens Schmidt wrote:
> And thanks for calling it "thoughtful".  So I take that as a principal
> confirmation that this idea is not completely off the tracks and worth
> more implementation effort.
Yes surely.
Personally I have not used hut's `graphql` function much and so I've not
spent much thought about how to best implement cursor handling there,
but the feature would surely help out a lot of people that are not as
comfortable with writing GraphQL queries.

> Good to hear that, thanks.  In [1] Simon raised some concerns about the
> server inadvertently getting DoS'ed, how do you plan to address that?
> Would you restrict the count parameter to some hard-coded maximum?
> Otherwise a simple "--count $(( 2**63 - 1 ))" would give you unlimited