Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.
Sign up📚 Docs: Guide - Hello World & Generative Shape Art #262
Conversation
0nkery
reviewed
Jan 23, 2018
| Test to make sure everything is correct by running `cargo run`. | ||
|
|
||
| Be patient. This can take a while on the first run. Subsequent builds will be much faster. | ||
| Be sure it says `Hello, world!` at the bottom. before continuing. |
This comment has been minimized.
This comment has been minimized.
0nkery
Jan 23, 2018
- at the bottom. before continuing.
+ at the bottom, before continuing.I'm not much into English grammar, but it feels like there's no need in comma at all.
This comment has been minimized.
This comment has been minimized.
dannyfritz
force-pushed the
dannyfritz:guide
branch
from
2b01084
to
84004a6
Jan 23, 2018
0nkery
reviewed
Jan 23, 2018
| 0. Updating state | ||
| 0. Drawing | ||
|
|
||
| `ggez` provides [`Trait ggez::Event::EventHandler`](https://docs.rs/ggez/0.4.0/ggez/event/trait.EventHandler.html) is the provided loop as a loop for us to use in our games. |
This comment has been minimized.
This comment has been minimized.
0nkery
Jan 23, 2018
•
ggez provides EventHandler as a loop ...
or
EventHandler is the provided loop for us to use ... (but I'd prefer former).
dannyfritz
force-pushed the
dannyfritz:guide
branch
2 times, most recently
from
5d521ab
to
8d51de2
Jan 23, 2018
This comment has been minimized.
This comment has been minimized.
0nkery
commented
Jan 23, 2018
|
Good start! I believe you'll make an awesome thing. |
dannyfritz
force-pushed the
dannyfritz:guide
branch
from
8d51de2
to
b4203e3
Jan 23, 2018
This comment has been minimized.
This comment has been minimized.
|
I like your roadmap. Things we need to think about from a project perspective (mostly just recording what we talked about on IRC):
|
termhn
reviewed
Jan 23, 2018
| 1. Updating state such as player position, health, etc. | ||
| 1. Drawing shapes, images, etc. | ||
|
|
||
| [`EventHandler`](https://docs.rs/ggez/0.4.0/ggez/event/trait.EventHandler.html) is a loop provided by `ggez` to use in our games. |
This comment has been minimized.
This comment has been minimized.
termhn
Jan 23, 2018
Contributor
I think this is better put as, "EventHandler is a way for us to interface with--and implement our own logic on top of--ggez's internal game loop." or something similar.
This comment has been minimized.
This comment has been minimized.
|
Sweet! I like the direction this is going and am definitely interested in helping out as it progresses! :D |
dannyfritz
force-pushed the
dannyfritz:guide
branch
2 times, most recently
from
2883ec1
to
b7c5c7f
Jan 26, 2018
dannyfritz
force-pushed the
dannyfritz:guide
branch
2 times, most recently
from
02bce41
to
34b41a1
Jan 26, 2018
This comment has been minimized.
This comment has been minimized.
itmuckel
commented
Feb 3, 2018
|
I like your style of writing (and I love emoticons). The only thing I'm missing is integration with specs-rs, but as soon as I'm proficient enough in this I consider contributing a part. (I'm still developing a game that makes excessive use of specs-rs). Also this is not too important. Regarding keeping it up to date: I think it's generally best-practice to have a PR contain changes to the docs as soon as it adds functionality or (more important) changes functionality. I think this is easier to follow, because changes to code and changes to docs are in the same commit/PR. |
dannyfritz
force-pushed the
dannyfritz:guide
branch
from
34b41a1
to
cd6bc22
Feb 12, 2018
Ealhad
suggested changes
Apr 13, 2018
Ealhad left a comment •
|
In HelloGgez.md, we create the state I think this is a nice guide! I especially like that you give some challenges/exercises at the end of each chapter. |
Ratysz
referenced this pull request
May 8, 2018
Open
A proper Getting Started guide might be nice? #190
This comment has been minimized.
This comment has been minimized.
|
@dannyfritz What's the status of this? Can we merge what's here so that other people can work on it? |
This comment has been minimized.
This comment has been minimized.
|
Yeah, you can merge this. I have a third one in the hopper I just need to format I can do another PR for later. Been going really slow as you can see. |
dannyfritz
changed the title
🚧 WIP DO NOT MERGE 🚧 🗣 Feedback Requested 🗣 📚 Docs: Guide
📚 Docs: Guide - Hello World & Generative Shape Art
May 9, 2018
icefoxen
merged commit f3592d6
into
ggez:master
May 9, 2018
This comment has been minimized.
This comment has been minimized.
|
Awesome, thank you! No worries. Perhaps we can do one PR per chapter in the future, to keep things easier to put together piece by piece. |
This comment has been minimized.
This comment has been minimized.
|
Hm, considering #311 do we really want to use glob imports in the guide? |
This comment has been minimized.
This comment has been minimized.
|
Nope! Good catch. |
This comment has been minimized.
This comment has been minimized.
Maybe it's better to give links without patch version? |
dannyfritz commentedJan 23, 2018
•
edited
Benefits of a Guide:
ggez's modules for deeper understandingAssumptions and Guidelines:
ggezwe/usperspective unless an action must betaken, thenyou/youris usedggezdocs when appropriateGuide Sections:
ggez! (Loop + Window + Conf)For another PR
ggez?