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.

Sign up for GitHub

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

Jump to bottom

Karl/stat tracker 2 #1084

Open
wants to merge 83 commits into
base: main
Choose a base branch
from
Open

Karl/stat tracker 2 #1084

wants to merge 83 commits into from

Commits on Jun 4, 2024

  1. created runner file with the code provided in iteration. Created Stat… 

    …_tracker file. Created a spec_helper file to hold SimpleCov. Installed Simplecov Gem.
    @James-Cochran
    James-Cochran committed Jun 4, 2024
    Configuration menu
    Copy the full SHA
    6eaa8c8 View commit details
    Browse the repository at this point in the history

  2. copied/pasted James code, added README.md project requirements

    @CoryBretsch
    CoryBretsch committed Jun 4, 2024
    Configuration menu
    Copy the full SHA
    36cea67 View commit details
    Browse the repository at this point in the history

  3. Merge pull request #1 from CoryBretsch/cory/set_up 

    copied/pasted James code, added README.md project requirements
    @SmilodonP
    SmilodonP authored Jun 4, 2024
    Configuration menu
    Copy the full SHA
    5e19c86 View commit details
    Browse the repository at this point in the history

  4. corrected merge conflicts

    @James-Cochran
    James-Cochran committed Jun 4, 2024
    Configuration menu
    Copy the full SHA
    f656314 View commit details
    Browse the repository at this point in the history

  5. Update README.md

    @CoryBretsch
    CoryBretsch authored Jun 4, 2024
    Configuration menu
    Copy the full SHA
    1901b29 View commit details
    Browse the repository at this point in the history

  6. Update README.md

    @CoryBretsch
    CoryBretsch authored Jun 4, 2024
    Configuration menu
    Copy the full SHA
    713727d View commit details
    Browse the repository at this point in the history

  7. Update .gitignore 

    fixed gitignore
    @SmilodonP
    SmilodonP authored Jun 4, 2024
    Configuration menu
    Copy the full SHA
    66fd83a View commit details
    Browse the repository at this point in the history

  8. created league.rb and league_spec.rb files. started the first test fo… 

    …r league initialize method.
    @James-Cochran
    James-Cochran committed Jun 4, 2024
    Configuration menu
    Copy the full SHA
    b52fd31 View commit details
    Browse the repository at this point in the history

  9. added Team class and spec that turns teams.csv into data objects

    @CoryBretsch
    CoryBretsch committed Jun 4, 2024
    Configuration menu
    Copy the full SHA
    04e2c4b View commit details
    Browse the repository at this point in the history

  10. Merge branch 'main' into cory/read_teams_data

    @James-Cochran
    James-Cochran authored Jun 4, 2024
    Configuration menu
    Copy the full SHA
    1bf7cd0 View commit details
    Browse the repository at this point in the history

  11. Merge pull request #2 from CoryBretsch/cory/read_teams_data 

    added Team class and spec that turns teams.csv into data objects
    @James-Cochran
    James-Cochran authored Jun 4, 2024
    Configuration menu
    Copy the full SHA
    4fe0669 View commit details
    Browse the repository at this point in the history

  12. corrected typo in league_spec and added csv require ot league

    @James-Cochran
    James-Cochran committed Jun 4, 2024
    Configuration menu
    Copy the full SHA
    34ecf5a View commit details
    Browse the repository at this point in the history

  13. resolved merge conflict in spec_helper.rb

    @James-Cochran
    James-Cochran committed Jun 4, 2024
    Configuration menu
    Copy the full SHA
    4c327f9 View commit details
    Browse the repository at this point in the history

  14. created game.rb file and game_spec file. created initial hash in the … 

    …game_spec, and the initialize in the game.rb
    @James-Cochran
    James-Cochran committed Jun 4, 2024
    Configuration menu
    Copy the full SHA
    572c7b2 View commit details
    Browse the repository at this point in the history

Commits on Jun 5, 2024

  1. filled out the last 2 questions in README

    @SmilodonP
    SmilodonP committed Jun 5, 2024
    Configuration menu
    Copy the full SHA
    d35c105 View commit details
    Browse the repository at this point in the history

  2. adding game_teams class and spec with test data file

    @CoryBretsch
    CoryBretsch committed Jun 5, 2024
    Configuration menu
    Copy the full SHA
    2e39e25 View commit details
    Browse the repository at this point in the history

  3. corrected game.rb file to games.rb. rewrote games class to be instanc… 

    …es instead of a hash. rewrote games_spec to correspond to games being instances.
    @James-Cochran
    James-Cochran committed Jun 5, 2024
    Configuration menu
    Copy the full SHA
    51d6254 View commit details
    Browse the repository at this point in the history

  4. Merge pull request #4 from CoryBretsch/cory/read_game_teams_data 

    adding game_teams class and spec with test data file
    @James-Cochran
    James-Cochran authored Jun 5, 2024
    Configuration menu
    Copy the full SHA
    e13ace6 View commit details
    Browse the repository at this point in the history

  5. started test for games_spec to verify it is pulling from CSV

    @James-Cochran
    James-Cochran committed Jun 5, 2024
    Configuration menu
    Copy the full SHA
    211b0db View commit details
    Browse the repository at this point in the history

  6. Merge branch 'main' into james_game_class

    @CoryBretsch
    CoryBretsch authored Jun 5, 2024
    Configuration menu
    Copy the full SHA
    44651ae View commit details
    Browse the repository at this point in the history

  7. Merge pull request #5 from CoryBretsch/james_game_class 

    James game class
    @CoryBretsch
    CoryBretsch authored Jun 5, 2024
    Configuration menu
    Copy the full SHA
    6817afb View commit details
    Browse the repository at this point in the history

  8. created games_test.csv for easier testing. Passing games_spec test to… 

    … pull data from the games.csv.
    @James-Cochran
    James-Cochran committed Jun 5, 2024
    Configuration menu
    Copy the full SHA
    3f2f5a5 View commit details
    Browse the repository at this point in the history

  9. Merge pull request #6 from CoryBretsch/james_game_class 

    created games_test.csv for easier testing. Passing games_spec test to…
    @SmilodonP
    SmilodonP authored Jun 5, 2024
    Configuration menu
    Copy the full SHA
    c80c5d0 View commit details
    Browse the repository at this point in the history

Commits on Jun 6, 2024

  1. added count_of_teams method

    @SmilodonP
    SmilodonP committed Jun 6, 2024
    Configuration menu
    Copy the full SHA
    605d93e View commit details
    Browse the repository at this point in the history

  2. Update README.md

    @CoryBretsch
    CoryBretsch authored Jun 6, 2024
    Configuration menu
    Copy the full SHA
    0e2b497 View commit details
    Browse the repository at this point in the history

  3. passing test for highest total score, lowest total score, percentage … 

    …of home wins, percentage of visitor wins and percentage of ties. Created matching methods for each test.
    @James-Cochran
    James-Cochran committed Jun 6, 2024
    Configuration menu
    Copy the full SHA
    c2da976 View commit details
    Browse the repository at this point in the history

  4. minimal progress on league stat methods

    @SmilodonP
    SmilodonP committed Jun 6, 2024
    Configuration menu
    Copy the full SHA
    1986644 View commit details
    Browse the repository at this point in the history

  5. YAY

    @CoryBretsch
    CoryBretsch committed Jun 6, 2024
    Configuration menu
    Copy the full SHA
    a9d836e View commit details
    Browse the repository at this point in the history

  6. Merge branch 'main' of github.com:CoryBretsch/futbol into cory/stat_t… 

    …racker_connection

no message necessary
    @CoryBretsch
    CoryBretsch committed Jun 6, 2024
    Configuration menu
    Copy the full SHA
    1fa987d View commit details
    Browse the repository at this point in the history

  7. Merge pull request #7 from CoryBretsch/cory/stat_tracker_connection 

    Cory/stat tracker connection
    @SmilodonP
    SmilodonP authored Jun 6, 2024
    Configuration menu
    Copy the full SHA
    2b18048 View commit details
    Browse the repository at this point in the history

  8. committing so that I can pull updated main

    @SmilodonP
    SmilodonP committed Jun 6, 2024
    Configuration menu
    Copy the full SHA
    d684678 View commit details
    Browse the repository at this point in the history

  9. committing so that I can pull updated main again

    @SmilodonP
    SmilodonP committed Jun 6, 2024
    Configuration menu
    Copy the full SHA
    8fd31ba View commit details
    Browse the repository at this point in the history

  10. passing test and method for count of games by season.

    @James-Cochran
    James-Cochran committed Jun 6, 2024
    Configuration menu
    Copy the full SHA
    0dccedd View commit details
    Browse the repository at this point in the history

  11. passing average goals per game with accompanying method

    @James-Cochran
    James-Cochran committed Jun 6, 2024
    Configuration menu
    Copy the full SHA
    2037353 View commit details
    Browse the repository at this point in the history

  12. Merge branch 'main' of github.com:CoryBretsch/futbol into karl/league… 

    …_statistics
    @SmilodonP
    SmilodonP committed Jun 6, 2024
    Configuration menu
    Copy the full SHA
    4bb4166 View commit details
    Browse the repository at this point in the history

  13. passing test with method for average goals by season.

    @James-Cochran
    James-Cochran committed Jun 6, 2024
    Configuration menu
    Copy the full SHA
    3f1eb3d View commit details
    Browse the repository at this point in the history

  14. resolved merge conflict.

    @James-Cochran
    James-Cochran committed Jun 6, 2024
    Configuration menu
    Copy the full SHA
    fe238f3 View commit details
    Browse the repository at this point in the history

  15. Merge pull request #8 from CoryBretsch/james_game_class 

    James game class
    @SmilodonP
    SmilodonP authored Jun 6, 2024
    Configuration menu
    Copy the full SHA
    1c2bee0 View commit details
    Browse the repository at this point in the history

Commits on Jun 7, 2024

  1. Update README.md

    @CoryBretsch
    CoryBretsch authored Jun 7, 2024
    Configuration menu
    Copy the full SHA
    1441103 View commit details
    Browse the repository at this point in the history

  2. moved all games_statistics methods into stat_tracker. Adjusted games_… 

    …statistics_spec to now test from stat_tracker. Commented out games_statistics methods.
    @James-Cochran
    James-Cochran committed Jun 7, 2024
    Configuration menu
    Copy the full SHA
    c43f907 View commit details
    Browse the repository at this point in the history

  3. Merge pull request #9 from CoryBretsch/james_game_class 

    moved all games_statistics methods into stat_tracker.  Adjusted games…
    @SmilodonP
    SmilodonP authored Jun 7, 2024
    Configuration menu
    Copy the full SHA
    cc76834 View commit details
    Browse the repository at this point in the history

  4. Merge branch 'main' of github.com:CoryBretsch/futbol into karl/league… 

    …_statistics
    @SmilodonP
    SmilodonP committed Jun 7, 2024
    Configuration menu
    Copy the full SHA
    36ca1f7 View commit details
    Browse the repository at this point in the history

Commits on Jun 8, 2024

  1. moved league stats into stat tracker, fixed formatting issues, and wo… 

    …rked on some methods
    @SmilodonP
    SmilodonP committed Jun 8, 2024
    Configuration menu
    Copy the full SHA
    e23a78b View commit details
    Browse the repository at this point in the history

  2. Delete coverage directory 

    Deleting cover directory - hoping this will allow the .gitignore file to work
    @CoryBretsch
    CoryBretsch authored Jun 8, 2024
    Configuration menu
    Copy the full SHA
    c8d692e View commit details
    Browse the repository at this point in the history

  3. made many minor adjustments to allow spec_harness to work.

    @CoryBretsch
    CoryBretsch committed Jun 8, 2024
    Configuration menu
    Copy the full SHA
    887dfa9 View commit details
    Browse the repository at this point in the history

  4. Update README.md

    @CoryBretsch
    CoryBretsch authored Jun 8, 2024
    Configuration menu
    Copy the full SHA
    1b1e2d4 View commit details
    Browse the repository at this point in the history

  5. Merge pull request #10 from CoryBretsch/cory/spec_harness 

    made many minor adjustments to allow spec_harness to work.
    @James-Cochran
    James-Cochran authored Jun 8, 2024
    Configuration menu
    Copy the full SHA
    13723eb View commit details
    Browse the repository at this point in the history

  6. committing to pull

    @SmilodonP
    SmilodonP committed Jun 8, 2024
    Configuration menu
    Copy the full SHA
    6a62e25 View commit details
    Browse the repository at this point in the history

  7. Merge branch 'main' of github.com:CoryBretsch/futbol into karl/league… 

    …_statistics
    @SmilodonP
    SmilodonP committed Jun 8, 2024
    Configuration menu
    Copy the full SHA
    26726d3 View commit details
    Browse the repository at this point in the history

  8. Merge pull request #12 from CoryBretsch/karl/league_statistics 

    Karl/league statistics
    @CoryBretsch
    CoryBretsch authored Jun 8, 2024
    Configuration menu
    Copy the full SHA
    acde812 View commit details
    Browse the repository at this point in the history

  9. Delete coverage directory

    @CoryBretsch
    CoryBretsch authored Jun 8, 2024
    Configuration menu
    Copy the full SHA
    97a1b96 View commit details
    Browse the repository at this point in the history

  10. delete coverage

    @CoryBretsch
    CoryBretsch committed Jun 8, 2024
    Configuration menu
    Copy the full SHA
    8eb4510 View commit details
    Browse the repository at this point in the history

  11. Merge pull request #13 from CoryBretsch/cory/delete_coverage 

    delete coverage
    @CoryBretsch
    CoryBretsch authored Jun 8, 2024
    Configuration menu
    Copy the full SHA
    35086ad View commit details
    Browse the repository at this point in the history

Commits on Jun 9, 2024

  1. adding high/low scoring away team with tests. passes with spec_harnes… 

    …s. Also added before(:each) block in stat_tracker_spec to clean things up
    @CoryBretsch
    CoryBretsch committed Jun 9, 2024
    Configuration menu
    Copy the full SHA
    c0ee2d8 View commit details
    Browse the repository at this point in the history

  2. Update README.md

    @CoryBretsch
    CoryBretsch authored Jun 9, 2024
    Configuration menu
    Copy the full SHA
    5275837 View commit details
    Browse the repository at this point in the history

  3. Merge pull request #14 from CoryBretsch/cory/high_low_scoring_visitor 

    adding high/low scoring away team with tests. passes with spec_harnes…
    @James-Cochran
    James-Cochran authored Jun 9, 2024
    Configuration menu
    Copy the full SHA
    4299a68 View commit details
    Browse the repository at this point in the history

  4. adding highest/lowest scoring home team with test. Spec harness passes

    @CoryBretsch
    CoryBretsch committed Jun 9, 2024
    Configuration menu
    Copy the full SHA
    e151145 View commit details
    Browse the repository at this point in the history

  5. Merge pull request #15 from CoryBretsch/cory/high_low_score_home 

    adding highest/lowest scoring home team with test. Spec harness passes
    @James-Cochran
    James-Cochran authored Jun 9, 2024
    Configuration menu
    Copy the full SHA
    069520f View commit details
    Browse the repository at this point in the history

  6. resolved merge conflict

    @James-Cochran
    James-Cochran committed Jun 9, 2024
    Configuration menu
    Copy the full SHA
    e946b60 View commit details
    Browse the repository at this point in the history

  7. added some league stats tests + skeletons

    @SmilodonP
    SmilodonP committed Jun 9, 2024
    Configuration menu
    Copy the full SHA
    93fafff View commit details
    Browse the repository at this point in the history

  8. commiting to pull from main

    @SmilodonP
    SmilodonP committed Jun 9, 2024
    Configuration menu
    Copy the full SHA
    c07a684 View commit details
    Browse the repository at this point in the history

  9. Merge branch 'main' of github.com:CoryBretsch/futbol into karl/stat_t… 

    …racker
    @SmilodonP
    SmilodonP committed Jun 9, 2024
    Configuration menu
    Copy the full SHA
    051c0eb View commit details
    Browse the repository at this point in the history

  10. created test and method for winningest coach

    @James-Cochran
    James-Cochran committed Jun 9, 2024
    Configuration menu
    Copy the full SHA
    d22083c View commit details
    Browse the repository at this point in the history

  11. Update README.md

    @James-Cochran
    James-Cochran authored Jun 9, 2024
    Configuration menu
    Copy the full SHA
    5e57dac View commit details
    Browse the repository at this point in the history

  12. no added function, but cleaned up code

    @SmilodonP
    SmilodonP committed Jun 9, 2024
    Configuration menu
    Copy the full SHA
    98d39f2 View commit details
    Browse the repository at this point in the history

  13. added best and worst offense methods with passing tests

    @SmilodonP
    SmilodonP committed Jun 9, 2024
    Configuration menu
    Copy the full SHA
    37768c6 View commit details
    Browse the repository at this point in the history

  14. Merge pull request #17 from CoryBretsch/james_season_branch 

    James season branch
    @SmilodonP
    SmilodonP authored Jun 9, 2024
    Configuration menu
    Copy the full SHA
    a841860 View commit details
    Browse the repository at this point in the history

  15. added most/fewest tackles with tests. passes with spec harness

    @CoryBretsch
    CoryBretsch committed Jun 9, 2024
    Configuration menu
    Copy the full SHA
    fc34fba View commit details
    Browse the repository at this point in the history

Commits on Jun 10, 2024

  1. added some comments

    @CoryBretsch
    CoryBretsch committed Jun 10, 2024
    Configuration menu
    Copy the full SHA
    c58c51d View commit details
    Browse the repository at this point in the history

  2. Merge branch 'main' into cory/most_fewest_tackles

    @CoryBretsch
    CoryBretsch authored Jun 10, 2024
    Configuration menu
    Copy the full SHA
    b22d4f6 View commit details
    Browse the repository at this point in the history

  3. Merge pull request #19 from CoryBretsch/cory/most_fewest_tackles 

    Cory/most fewest tackles
    @CoryBretsch
    CoryBretsch authored Jun 10, 2024
    Configuration menu
    Copy the full SHA
    a63b229 View commit details
    Browse the repository at this point in the history

  4. Merge branch 'main' into karl/stat_tracker

    @CoryBretsch
    CoryBretsch authored Jun 10, 2024
    Configuration menu
    Copy the full SHA
    ab68464 View commit details
    Browse the repository at this point in the history

  5. Update stat_tracker_spec.rb

    @CoryBretsch
    CoryBretsch authored Jun 10, 2024
    Configuration menu
    Copy the full SHA
    c4bc850 View commit details
    Browse the repository at this point in the history

  6. Update stat_tracker.rb

    @CoryBretsch
    CoryBretsch authored Jun 10, 2024
    Configuration menu
    Copy the full SHA
    8a16029 View commit details
    Browse the repository at this point in the history

  7. resolved merge conflict.

    @James-Cochran
    James-Cochran committed Jun 10, 2024
    Configuration menu
    Copy the full SHA
    75047e2 View commit details
    Browse the repository at this point in the history

  8. Merge branch 'main' into karl/stat_tracker

    @CoryBretsch
    CoryBretsch authored Jun 10, 2024
    Configuration menu
    Copy the full SHA
    df26182 View commit details
    Browse the repository at this point in the history

  9. Merge pull request #18 from CoryBretsch/karl/stat_tracker 

    Karl/stat tracker
    @CoryBretsch
    CoryBretsch authored Jun 10, 2024
    Configuration menu
    Copy the full SHA
    8044a46 View commit details
    Browse the repository at this point in the history

  10. Update stat_tracker_spec.rb

    @CoryBretsch
    CoryBretsch authored Jun 10, 2024
    Configuration menu
    Copy the full SHA
    3f52ca5 View commit details
    Browse the repository at this point in the history

  11. added most/least accurate code. tests do not pass

    @SmilodonP
    SmilodonP committed Jun 10, 2024
    Configuration menu
    Copy the full SHA
    eee49df View commit details
    Browse the repository at this point in the history

  12. attempts to correct winningest coach method.

    @James-Cochran
    James-Cochran committed Jun 10, 2024
    Configuration menu
    Copy the full SHA
    196c83d View commit details
    Browse the repository at this point in the history

  13. resove merge conflict.

    @James-Cochran
    James-Cochran committed Jun 10, 2024
    Configuration menu
    Copy the full SHA
    44e2516 View commit details
    Browse the repository at this point in the history

  14. Merge pull request #20 from CoryBretsch/james_season_branch 

    James season branch
    @CoryBretsch
    CoryBretsch authored Jun 10, 2024
    Configuration menu
    Copy the full SHA
    4b54a20 View commit details
    Browse the repository at this point in the history

  15. recommitting after merge

    @SmilodonP
    SmilodonP committed Jun 10, 2024
    Configuration menu
    Copy the full SHA
    3218046 View commit details
    Browse the repository at this point in the history