search

exilon / QuickLogger

Powerful and flexible library for logging on files, console, memory, email, rest, eventlog, syslog, slack, telegram, redis, logstash, elasticsearch, influxdb, graylog, Sentry, Twilio, ide debug messages and throw events for Delphi/Firemonkey/freepascal/.NET (Windows/Linux/OSX/IOS/Android).
Apache License 2.0
381 stars 84 forks source link

New Provider for StringLists #88

Closed jfudickar closed 1 year ago

jfudickar commented 1 year ago

Hi, can I get access to the repository that I can create a branch and pull request for a new string list provider. Otherwise I can put the file here directly.

Regards Jens

luebbe commented 1 year ago

That is not necessary. You can fork the repository and create a PR from your fork. Then the project owner is able to review and merge it.

jfudickar commented 1 year ago

I have to look into this. I never worked with forks before.

luebbe commented 1 year ago

It's the easiest way to go and doesn't require any administrative effort by the project maintainer. Just click on "Fork" at the top right of this repository and you create a fork of it in your personal work area. Write your code, test and commit it to your fork. Once you're done you will see that "your fork is xxx revisions ahead of ..." and an active "create pull request" button in your fork. Do just this and from there on its up to the maintainer of this repository to merge your PR.

jfudickar commented 1 year ago

Did it that way. Thanks for the hint.