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.

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

Add auto-release #373

Closed
mtrezza opened this issue May 20, 2024 · 4 comments · Fixed by #375
Closed

Add auto-release #373

mtrezza opened this issue May 20, 2024 · 4 comments · Fixed by #375
Labels

Comments

@mtrezza
Copy link
Member

mtrezza commented May 20, 2024

Let's add auto-release to this repo. Even though there hasn't been much activity in the last couple of years, but it may help to motivate contributors to submit PR if they know we can merge and release fast.

Copy link

Thanks for opening this issue!

@mtrezza mtrezza pinned this issue May 20, 2024
@mtrezza mtrezza linked a pull request May 20, 2024 that will close this issue
@mtrezza mtrezza reopened this May 23, 2024
@mtrezza
Copy link
Member Author

mtrezza commented May 23, 2024

@Werkezeugs could you please check out the latest release 3.0 that we've just published? I want to make sure it worked fine, before I merge your PR.

@mtrezza mtrezza closed this as completed May 24, 2024
@mtrezza mtrezza unpinned this issue May 24, 2024
@Werkezeugs
Copy link
Contributor

Hi @mtrezza yes, seems to work fine on my project now :)

In the meantime I found another bug related to this. The .Include() does not work as expected for anonym users.
Will create a seperate PR for this ;)

Thanks for the fast reply

@mtrezza
Copy link
Member Author

mtrezza commented May 25, 2024

Great, I already reviewed your PR and sent you some feedback - thanks for testing it out!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants