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

Ability to configure the config handle #20

Merged
merged 12 commits into from
Dec 2, 2024
Merged

Conversation

edalzell
Copy link
Owner

Closes #14

@afonic
Copy link

afonic commented Oct 9, 2024

Hi @edalzell, will this be merged? It would greatly simplify stuff. :)

@edalzell
Copy link
Owner Author

edalzell commented Oct 9, 2024

Hi @edalzell, will this be merged? It would greatly simplify stuff. :)

I can look, it's been a while lol.

@afonic
Copy link

afonic commented Oct 26, 2024

👀

@edalzell
Copy link
Owner Author

👀

Sorry, been dealing with stuff and super busy. I'll get there.

@afonic
Copy link

afonic commented Nov 1, 2024

No worries, I just saw you merged back main and I thought you forgot to merge the PR.

@edalzell
Copy link
Owner Author

edalzell commented Dec 1, 2024

No worries, I just saw you merged back main and I thought you forgot to merge the PR.

@afonic Can you give this a try and let me know, I think this should be good to go

@afonic
Copy link

afonic commented Dec 2, 2024

@edalzell just tried this PR in Resrv, it works just fine, thanks!

@edalzell edalzell merged commit 9203f46 into main Dec 2, 2024
1 check passed
@edalzell edalzell deleted the feature/configure-handle branch December 2, 2024 20:24
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 this pull request may close these issues.

Allow using a different config file name from package name
2 participants