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

Contributing #1

Closed
czifro opened this issue May 8, 2018 · 6 comments
Closed

Contributing #1

czifro opened this issue May 8, 2018 · 6 comments

Comments

@czifro
Copy link

czifro commented May 8, 2018

I'd like to offer my support as a contributor to help maintain, address bugs, and potentially add features to it in any spare time I have. Mausch had some interesting ideas that could improve usability: mausch#10, mausch#8, etc. There are some apparent bugs as well: mausch#28, mausch#13, etc. I would be willing to address some of these when I get time.

I see in your PR you mentioned not having a Linux environment to test off of. The closest I have to that is MacOS. Unless it is something very specific to Linux, I believe I can help with more general *nix testing.

Lastly, thanks for porting this to NET Core 2.0, it has been a tremendous help with my work.

Cheers!

@irium
Copy link
Owner

irium commented May 9, 2018

Thank you! Feel free to fix any bugs and send any PRs you want. I'm not a real owner, nor a maitainer of the repo. I just have seen this project and found that this is what I need, but on .NET Core. So I just ported it (this wasn't too hard to be honest :) and published to NuGet after @haf asked me.

First of all I recommend to port @haf 's changes about namespaces to here.
See commits from https://github.com/haf/FsSql/tree/feature/namespaces-logging branch.

@czifro
Copy link
Author

czifro commented May 9, 2018

That seems simple enough to get started. I also think that the build process could be improved. Docker could be used to run the FAKE script so that there is a consistent environment to run tests and create packages, which could be helpful for using a CI system.

@haf
Copy link

haf commented May 9, 2018

Feel free to take what you need from @SuaveIO and @logary's build scripts

@czifro
Copy link
Author

czifro commented May 9, 2018

Thanks for the references.

@czifro czifro closed this as completed May 9, 2018
@irium
Copy link
Owner

irium commented May 9, 2018

@czifro Please don't make Docker the only way to run tests :)

@czifro
Copy link
Author

czifro commented May 9, 2018

You got it! I will just set it up so that it is an option. So to recap, there will be the following methods:

  1. build.dat
  2. build.sh -- needs adjustments
  3. docker-build.bat -- may need help with testing the script, I don't have a Windows machine
  4. docker-build.sh

If this seems ok, I can work on it in my spare time.

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

No branches or pull requests

3 participants