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

Beta test m2l3 #3

Closed
wants to merge 4 commits into from
Closed

Beta test m2l3 #3

wants to merge 4 commits into from

Conversation

mktoma728
Copy link

Updated README.md file with the following changes:

Updated the README.md file with the following changes:

  1. Indented the code/text blocks in the Getting Started Section to resolve
    the numbered list sequence issue
  2. Updated the file hierachy diagram and associated narrative to reference
    the student-start directory (as done in lesson1)

Note: I did not make any of these editorial / non-format changes in the HTML readme. If these changes are approved, I made the assumption that pandoc will be used to propagate the changes. Please let me know if that is a bad assumption

thanks,
Mike

Changes to be committed:
modified:   README.md

1. Numbered Lists within each section were broken due to code blocks not being nested

2. Updated `Change the source code ...` statement to specify the changes are to
   enable a case statement to be used, as opposed to changes to make the conditional
   logic to be `true`

3. For completeness, added the `.rspec` file in the file hierachy tree diagram

4. Re-ordered the bulleted list to follow the sequence of the file hierachy tree diagram

5. Added a 4th `F` to `.FFF` in the rspec code block, since there are 4 tests

6. Updated file name from case_spec to lesson1_spec in the rspec code block

7. Added another elsif clause failed example since (2) are tested with the rspec

8. Updated successful rspec code block to state 5 examples, instead of 4
Changes to be committed:
modified:   README.md

1. Updated file hierachy tree to specify .rspec is a hidden file
Changes to be committed:
modified:   ../README.md

1. Indented Code block to correct numbered list issue

2. Updated grammatical errors as part of Item2 under Technical Requirements
…solve

   the numbered list sequence issue

2. Updated the file hierachy diagram and associated narrative to reference
   the student-start directory (as done in lesson1)
JSierawski added a commit to JSierawski/fullstack-course1-module2 that referenced this pull request Jun 7, 2017
…sera#3: Classes

The overall goal of this assignment is to implement a Ruby class
with - class attributes, class methods, initializer methods, instance
attributes, instance methods.

The functional goal of the assignment is to write a Person class
that will maintain state thru instance and class attributes and
provide access thru instance and class class methods.
andre-lfa pushed a commit to andre-lfa/fullstack-course1-module2 that referenced this pull request Jul 17, 2018
@kanzn kanzn mentioned this pull request Jul 25, 2018
@kalmanh kalmanh closed this Oct 14, 2018
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

Successfully merging this pull request may close these issues.

None yet

2 participants