contributing

NOTICE: This forum is archived as read only.
Please use the Github Discussions at https://github.com/exult/exult/discussions
Forum rules
NOTICE: This forum is archived as read only.
Please use the Github Discussions at https://github.com/exult/exult/discussions
Locked
alagner
Posts: 118
Joined: Thu May 14, 2020 1:34 pm

contributing

Post by alagner »

Just to make things clear: I've finally found some time to fiddle around with Exult's code.
At the moment I going through the bugtracker, for example I think I managed to implement a fix for this one:
https://sourceforge.net/p/exult/bugs/1936/
What should I do in such case? Register on SF and provide a patch as an attachment?

If I haven't RTFMed enough then sorry for introducing mess here, just please point me to the right place :)

BTW, nice to see the project still alive after all these years.
A.M.Miera
vel. Alagner Dragon
Dominus
Site Admin
Posts: 5656
Joined: Thu May 14, 2020 1:34 pm

Re: contributing

Post by Dominus »

Thanks a lot!
Best would be if you make a sourceforge account and provide a patch. That is how we used to operate.
But since we moved the code to Github, a pull request is probably as welcome. We haven't really established "rules" for this since the move to github ;)
So for now, either is fine ;)
--
Read the documentation and the FAQ! There is no excuse for not reading them! RTFM
Read the Rules!
We do not support Piracy/Abandonware/Warez!
alagner
Posts: 118
Joined: Thu May 14, 2020 1:34 pm

Re: contributing

Post by alagner »

Ok, done.

BTW, pull requests seem much more "gitish" (and thus consistent with the whole repo) way, I think I'll try those when commiting next code (unless you'll prefer it otherwise).

Have a nice weekend :)
A.M.Miera
vel. Alagner Dragon
Dominus
Site Admin
Posts: 5656
Joined: Thu May 14, 2020 1:34 pm

Re: contributing

Post by Dominus »

Thanks again! I guess we will need to try things out and see what we prefer in the end. So yes, please do a pull request next time, so we can decide ;)
But I don't see any problem either way. With a patch we have it stored with the bug, and need to apply it and then close the bug with a commit reference.
With a pull request (that links to the bug) we need to merge and close the bug report with a commit reference...
Merge might be nicer since ut properly references your contribution and points at your fork.
--
Read the documentation and the FAQ! There is no excuse for not reading them! RTFM
Read the Rules!
We do not support Piracy/Abandonware/Warez!
nadir
Site Admin
Posts: 407
Joined: Thu May 14, 2020 1:34 pm

Re: contributing

Post by nadir »

+1 on Pull Requests
Locked