-
Notifications
You must be signed in to change notification settings - Fork 20
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
include domain name in module paths; local module paths may be reserved in the future #4
Comments
Hi Robin, Thanks for trying it out! I'm pretty new to Go and haven't really got my head around the new module system. I'll make the changes you've mentioned later this weekend. If you're interested in documentation I have another repository that I'll be using for more in depth documentation. https://github.com/JetSetIlly/Gopher2600-Dev-Docs Not much in there at the moment but I'll be adding to it next week. Somebody has expressed an interest in using the emulator in a machine learning project so I need to document the controller system and all that kind of stuff. Thanks again. |
No worries :) We're all here to learn and improve. The change I mentioned would look something like #5. |
Yeah, I saw that too. Would be really cool! Coincidentally, I've also been looking at reinforcement learning these last few days, and was thus excited to see gold repo of @aunum as referenced from the Reddit thread.
Thanks for putting the docs online! It's great to have both API level documentation and high-level documentation. Cheers, |
First of, really cool work @JetSetIlly! I tried running Gopher2600 on two games and it worked well :)
As I wanted to dive into the documentation of the project, I checked the GoDoc page. At the end of the page, I noticed these warnings:
If possible, I'd suggest updating the module path to
github.com/jetsetilly/gopher2600
, as this includes the domain name in the fully qualified import path. One reason this is important is that there is a proposal to reserve specific path prefixes for local (user-defined) modules (e.g.local/gopher2600
); see golang/go#37641. Since the Gopher2600 project is hosted on a remote site, it makes sense to include the full domain name in the module path.Wish you all the best and happy coding.
Cheers,
Robin
The text was updated successfully, but these errors were encountered: