John Olheiser jolheiser
Loading Heatmap…

jolheiser pushed to master at gitea/website

  • debaed0c07 Add reminder about the location of the themes repository (#108)

2 weeks ago

jolheiser merged pull request gitea/website#108

README: add reminder about the location of the themes repository

2 weeks ago

jolheiser commented on pull request gitea/go-sdk#589

Add function to get commit diff/patch

I think the difference is that the "generic" func is non-exported in the PR case. I'm not against that approach, mostly I think the _exported_ funcs being explicitly named is the design…

2 weeks ago

jolheiser opened issue gitea/distea#8

Add fly.io preview deploy command

2 weeks ago

jolheiser pushed to jolheiser-patch-1 at gitea/awesome-gitea

3 weeks ago

jolheiser commented on issue gitea/awesome-gitea#77

Woodpecker's "see docs" link returns a 404

I've sent a PR to rectify, but in the meantime the URL is https://woodpecker-ci.org/docs/administration/forges/gitea/ 🙂

3 weeks ago

jolheiser created pull request gitea/awesome-gitea#78

Fix woodpecker link

3 weeks ago

jolheiser pushed to jolheiser-patch-1 at gitea/awesome-gitea

3 weeks ago

jolheiser created branch jolheiser-patch-1 in gitea/awesome-gitea

3 weeks ago

jolheiser pushed to 1.16.7 at jolheiser/blog

3 weeks ago

jolheiser created pull request gitea/blog#193

Blog 1.16.7

3 weeks ago

jolheiser created branch 1.16.7 in jolheiser/blog

3 weeks ago

jolheiser pushed to 1.16.7 at jolheiser/blog

3 weeks ago

jolheiser commented on pull request gitea/go-sdk#589

Add function to get commit diff/patch

I wonder if two funcs would make sense. One for diff, one for patch?

3 weeks ago

jolheiser commented on pull request gitea/awesome-gitea#76

Add Gitea Actions link

So is this used in conjunction with a push mirror? Do you have an example of this running somewhere publicly? I'm fairly sure I know how it works, but not sure what kicks it off, since this…

3 weeks ago

jolheiser merged pull request gitea/go-sdk#570

Allow PR review with comments only

3 weeks ago

jolheiser pushed to master at gitea/go-sdk

  • 319a978c6c Allow PR review with comments only (#570)

3 weeks ago

jolheiser commented on pull request gitea/go-sdk#570

Allow PR review with comments only

Could you update your branch? After that CI should be able to run. 🙂

3 weeks ago

jolheiser deleted branch resp-err from jolheiser/go-sdk

3 weeks ago

jolheiser deleted branch commit-api from jolheiser/go-sdk

3 weeks ago

jolheiser commented on pull request gitea/go-sdk#587

Enforce golangci-lint + gofumpt

> > Odd, it _was_ passing. Looks like a sum error now? > > It was a lie, the binaries weren't downloaded(see the `... not found` errors). And now it's downloading and it's giving a sum error…

3 weeks ago

jolheiser commented on pull request gitea/go-sdk#587

Enforce golangci-lint + gofumpt

Odd, it _was_ passing. Looks like a sum error now?

3 weeks ago

jolheiser merged pull request gitea/go-sdk#586

Add the Accept/Reject transfer of a repository.

4 weeks ago

jolheiser pushed to master at gitea/go-sdk

  • ad3580e44d Add the Accept/Reject transfer of a repository. (#586)

4 weeks ago

jolheiser commented on pull request gitea/go-sdk#587

Enforce golangci-lint + gofumpt

Finally, CI failures relevant.

4 weeks ago

jolheiser commented on pull request gitea/go-sdk#587

Enforce golangci-lint + gofumpt

Hmm...I'm not convinced that `Validate` should modify the struct, though that's largely personal preference. I also agree `(&opt).Validate()` looks a bit silly, but from an API standpoint we…

4 weeks ago

jolheiser commented on pull request gitea/go-sdk#587

Enforce golangci-lint + gofumpt

Oof, I'm not (personally) a fan of side-effects like that. I would think defaults should be set prior to `Validate` rather than `Validate` changing the struct. I feel like passing in a…

4 weeks ago