{"action":"created","issue":{"url":"https://api.github.com/repos/Ada-C8/Solar-System/issues/22","repository_url":"https://api.github.com/repos/Ada-C8/Solar-System","labels_url":"https://api.github.com/repos/Ada-C8/Solar-System/issues/22/labels{/name}","comments_url":"https://api.github.com/repos/Ada-C8/Solar-System/issues/22/comments","events_url":"https://api.github.com/repos/Ada-C8/Solar-System/issues/22/events","html_url":"https://github.com/Ada-C8/Solar-System/pull/22","id":250541721,"number":22,"title":"Pipes - Sairagul -Solar_System","user":{"login":"sairagula","id":22013655,"avatar_url":"https://avatars3.githubusercontent.com/u/22013655?v=4","gravatar_id":"","url":"https://api.github.com/users/sairagula","html_url":"https://github.com/sairagula","followers_url":"https://api.github.com/users/sairagula/followers","following_url":"https://api.github.com/users/sairagula/following{/other_user}","gists_url":"https://api.github.com/users/sairagula/gists{/gist_id}","starred_url":"https://api.github.com/users/sairagula/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sairagula/subscriptions","organizations_url":"https://api.github.com/users/sairagula/orgs","repos_url":"https://api.github.com/users/sairagula/repos","events_url":"https://api.github.com/users/sairagula/events{/privacy}","received_events_url":"https://api.github.com/users/sairagula/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2017-08-16T07:57:37Z","updated_at":"2017-08-18T17:09:46Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Ada-C8/Solar-System/pulls/22","html_url":"https://github.com/Ada-C8/Solar-System/pull/22","diff_url":"https://github.com/Ada-C8/Solar-System/pull/22.diff","patch_url":"https://github.com/Ada-C8/Solar-System/pull/22.patch"},"body":"# Solar System\r\nCongratulations! You're submitting your assignment.\r\n\r\n## Comprehension Questions\r\n\r\n|  Question \t|  Answer \t|\r\n|---\t|---\t|\r\n|  What was the purpose of the `initialize` method in your class?\t|  To define my instance variables  |\r\n|  Describe an instance variable you used and what you used it for.\t|   I used @planets instance variable to store an array of planets created from my Planet class. Inside my Solarsystem class I used it in several methods \t|\r\n|  Describe what the difference would be if your `SolarSystem` used an `Array` vs a `Hash`. \t| I used array to store my planet objects. I had a method in my Planet class to print out in a given way, if I used hash I couldn't be able to use it\t|\r\n|  Do you feel like you used consistent indentation throughout your code? \t|   Mostly yes,  sometimes I used auto indentation\t|\r\n"},"comment":{"url":"https://api.github.com/repos/Ada-C8/Solar-System/issues/comments/323409302","html_url":"https://github.com/Ada-C8/Solar-System/pull/22#issuecomment-323409302","issue_url":"https://api.github.com/repos/Ada-C8/Solar-System/issues/22","id":323409302,"user":{"login":"sairagula","id":22013655,"avatar_url":"https://avatars3.githubusercontent.com/u/22013655?v=4","gravatar_id":"","url":"https://api.github.com/users/sairagula","html_url":"https://github.com/sairagula","followers_url":"https://api.github.com/users/sairagula/followers","following_url":"https://api.github.com/users/sairagula/following{/other_user}","gists_url":"https://api.github.com/users/sairagula/gists{/gist_id}","starred_url":"https://api.github.com/users/sairagula/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sairagula/subscriptions","organizations_url":"https://api.github.com/users/sairagula/orgs","repos_url":"https://api.github.com/users/sairagula/repos","events_url":"https://api.github.com/users/sairagula/events{/privacy}","received_events_url":"https://api.github.com/users/sairagula/received_events","type":"User","site_admin":false},"created_at":"2017-08-18T17:09:46Z","updated_at":"2017-08-18T17:09:46Z","body":"Awesome, thank you!!!\n\nOn Thu, Aug 17, 2017 at 3:35 PM, Jamie Pilgrim <notifications@github.com>\nwrote:\n\n> Solar System What We're Looking For\n> Feature Feedback\n> Created Custom Class with initialize method & instance variables. Well\n> Done\n> Used an Array to store a list of planets in the SolarSystem class. Well\n> Done\n> Readable code with consistent indentation. Well Done\n> Created a pull request with your name & the template questions answered. Well\n> Done - initialize can also call other methods.\n> Submission meets project expectations.\n> *Additional Feedback*\n> Naming Conventions class names should be CamelCase, with each word\n> capitalized. instead of Solarsystem, it should be SolarSystem\n> helper method Instead of writing attr_reader for each attribute, you can\n> do it all on one line, by separating each attribute with a comma, like so attr_reader\n> :name, :mass, :diameter, :year_length, :distance_from_sun\n> list_planets method When needing a counter number and working with an\n> entire array, using the .each_with_index method on the array will iterate\n> through the array *and* give you the index number of the element.\n> distance_from_other_planets method Its doing something. As long as you\n> have two planet instances, you can call it like this: puts\n> p1.distance_from_other_planet(p2). I commented out the user interface\n> portion and ran this.\n>\n> —\n> You are receiving this because you authored the thread.\n> Reply to this email directly, view it on GitHub\n> <https://github.com/Ada-C8/Solar-System/pull/22#issuecomment-323212674>,\n> or mute the thread\n> <https://github.com/notifications/unsubscribe-auth/AU_m1wlTU-XZbvcdAPnk0nSOdwc9cPKcks5sZMAzgaJpZM4O4jSy>\n> .\n>\n"}}
{"actor":{"display_login":"sairagula"}}