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

3604 - 0 #118

Closed
wants to merge 3 commits into from
Closed

3604 - 0 #118

wants to merge 3 commits into from

Conversation

MargaretOm
Copy link
Contributor

Номер

3604

Номер задания

0

Ссылка на видео с демо

https://drive.google.com/open?id=1nVRCmfb5ADNbwfTuiJ7vMwaAtlcWALHq

Комментарии

All done, and I believe that is correct

hound[bot]
hound bot reviewed Jul 9, 2019
View reviewed changes
3604/0/about_sandwich_code.rb
file = File.open(file_name)
yield(file)
ensure
file.close if file
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style/SafeNavigation: Use safe navigation (&.) instead of checking if an object exists before calling the method.

3604/0/about_sandwich_code.rb
return line if line =~ /e/
end
ensure
file.close if file
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style/SafeNavigation: Use safe navigation (&.) instead of checking if an object exists before calling the method.

3604/0/about_sandwich_code.rb
count += 1 while file.gets
count
ensure
file.close if file
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style/SafeNavigation: Use safe navigation (&.) instead of checking if an object exists before calling the method.

@MargaretOm MargaretOm closed this Jul 9, 2019
@MargaretOm MargaretOm reopened this Jul 9, 2019
@MargaretOm MargaretOm closed this Jul 9, 2019
mikitsik pushed a commit to mikitsik/summer-2019 that referenced this pull request Jul 15, 2019
@anatoliliotych
# Это объединение 140 коммитов.
2ce071a 
# Это 1-е сообщение коммита:

Add the very very first homework task

# Это сообщение коммита номер rubizza-camp#2:

new rubocop config

# Это сообщение коммита номер rubizza-camp#3:

Let's 2019 begin

# Это сообщение коммита номер rubizza-camp#4:

Resolve reek issue with wrong config format

# Это сообщение коммита номер rubizza-camp#5:

Uploading source code files

# Это сообщение коммита номер rubizza-camp#6:

Deleting unnecessary file

# Это сообщение коммита номер rubizza-camp#7:

Correcting ruby styles

# Это сообщение коммита номер rubizza-camp#8:

Initial commit

# Это сообщение коммита номер rubizza-camp#9:

Fix for rubocop

# Это сообщение коммита номер rubizza-camp#10:

Fix for hound

# Это сообщение коммита номер rubizza-camp#11:

Fixing for hound and refactoring DiceSet

# Это сообщение коммита номер rubizza-camp#12:

Add homework 3715 for task 0

# Это сообщение коммита номер rubizza-camp#13:

Fix issues indicated by the hound in homework 0

  To pass the Hound and match the human style code.

# Это сообщение коммита номер rubizza-camp#14:

Create file_to_delete.md
# Это сообщение коммита номер rubizza-camp#15:

3600 - 0

All files are checked via rubocop and reek
# Это сообщение коммита номер rubizza-camp#16:

Delete file_to_delete.md
# Это сообщение коммита номер rubizza-camp#17:

Create neo.rb
# Это сообщение коммита номер rubizza-camp#18:

Delete neo.rb
# Это сообщение коммита номер rubizza-camp#19:

Moved files to another folder

# Это сообщение коммита номер rubizza-camp#20:

Completed all ruby koans

# Это сообщение коммита номер rubizza-camp#21:

Fix most cod style for rubocop

# Это сообщение коммита номер rubizza-camp#22:

Fix some rubocop complaints

# Это сообщение коммита номер rubizza-camp#23:

Fix all koans indicated by the hound

# Это сообщение коммита номер rubizza-camp#24:

Renamed the latest commit

# Это сообщение коммита номер rubizza-camp#25:

Correct the error in triangle.rb

# Это сообщение коммита номер rubizza-camp#26:

Correct the alignments

# Это сообщение коммита номер rubizza-camp#27:

Solve half of Koans

# Это сообщение коммита номер rubizza-camp#28:

Solved all Koans

# Это сообщение коммита номер rubizza-camp#29:

Fix part of codestyle issues

# Это сообщение коммита номер rubizza-camp#30:

Fix half of codestyle issues

# Это сообщение коммита номер rubizza-camp#31:

Fix all codestyle issues

# Это сообщение коммита номер rubizza-camp#32:

Fix codestyle with new config

Update config files

Update config files

Update config files

Update config files

# Это сообщение коммита номер rubizza-camp#33:

Applied requested changes

# Это сообщение коммита номер rubizza-camp#34:

Fix about_control_statments

# Это сообщение коммита номер rubizza-camp#35:

Removed unnecessary rubocop:disables

# Это сообщение коммита номер rubizza-camp#36:

resloved koans and fixed rubocop

# Это сообщение коммита номер rubizza-camp#37:

fixed

# Это сообщение коммита номер rubizza-camp#38:

fix

# Это сообщение коммита номер rubizza-camp#39:

fix

# Это сообщение коммита номер rubizza-camp#40:

Fixed half reek's errors

# Это сообщение коммита номер rubizza-camp#41:

finished fix reek errors

# Это сообщение коммита номер rubizza-camp#42:

finished fix rubocop warnings

# Это сообщение коммита номер rubizza-camp#43:

fixed 'positive' warninig

# Это сообщение коммита номер rubizza-camp#44:

removed parentheses around a method call

# Это сообщение коммита номер rubizza-camp#45:

Come back koens's code and disable rubocop checker

# Это сообщение коммита номер rubizza-camp#46:

fixed variable name in test_methods_can_take_an_explicit_block_argument

# Это сообщение коммита номер rubizza-camp#47:

Edit method's arguments

# Это сообщение коммита номер rubizza-camp#48:

edit answer in test_methods_can_see_if_they_have_been_called_with_a_block
use array.uniq in triangle.rb

# Это сообщение коммита номер rubizza-camp#49:

fixed rubocop's error

# Это сообщение коммита номер rubizza-camp#50:

simplify code(use case then 3 methods) and fixed wrong call enumerable method in check_below_zero_triagle

# Это сообщение коммита номер rubizza-camp#51:

simplify calling errors check and removed triangle_analyzes

# Это сообщение коммита номер rubizza-camp#52:

merge conditions and rename triangle_valud? into triangle_validate?

# Это сообщение коммита номер rubizza-camp#53:

delete redundant result

# Это сообщение коммита номер rubizza-camp#54:

fixed rubocop warning

# Это сообщение коммита номер rubizza-camp#55:

use new instance of array

# Это сообщение коммита номер rubizza-camp#56:

fixed space's warning

# Это сообщение коммита номер rubizza-camp#57:

init

# Это сообщение коммита номер rubizza-camp#58:

fix Style/StringLiterals errors in scope and sandwich

# Это сообщение коммита номер rubizza-camp#59:

fix Style/StringLiterals errors in sandwich

# Это сообщение коммита номер rubizza-camp#60:

fix Style/StringLiterals and Style/SymbolArray errors in scope triangle path

# Это сообщение коммита номер rubizza-camp#61:

fix Style/StringLiterals errors in variable_scope

# Это сообщение коммита номер rubizza-camp#62:

fix Style/StringLiterals and Style/SafeNavigation errors in sandwich and to_str

# Это сообщение коммита номер rubizza-camp#63:

fix Style/SafeNavigation and Style/SymbolProc errors in proxy and symbols

# Это сообщение коммита номер rubizza-camp#64:

disable rubocop and rewrite code in ctrl_statements

# Это сообщение коммита номер rubizza-camp#65:

disable rubocop in neo

# Это сообщение коммита номер rubizza-camp#66:

fix Style/WordArray and Style/StringLiterals errors in neo versioning strings

# Это сообщение коммита номер rubizza-camp#67:

fix Style/StringLiterals errors in module objects sandwich

# Это сообщение коммита номер rubizza-camp#68:

fix Style/StringLiterals errors and add empty lines in strings symbols

# Это сообщение коммита номер rubizza-camp#69:

delete unnecessary empty lines and fix Style/StringLiterals symbols triangle var_scope

# Это сообщение коммита номер rubizza-camp#70:

rewrite all tests

# Это сообщение коммита номер rubizza-camp#71:

disable rubocop and delete empty lines

# Это сообщение коммита номер rubizza-camp#72:

change rubocop config for fix Layout/EndOfLine

# Это сообщение коммита номер rubizza-camp#73:

change rubocop config and disable

# Это сообщение коммита номер rubizza-camp#74:

try fix Layout/EndOfLine error and enable rubocop

# Это сообщение коммита номер rubizza-camp#75:

try fix Layout/EndOfLine error

# Это сообщение коммита номер rubizza-camp#76:

test Layout/EndOfFile

# Это сообщение коммита номер rubizza-camp#77:

disable rubocop and fix Layout/EndOfLine error

# Это сообщение коммита номер rubizza-camp#78:

disable and enable rubocop

# Это сообщение коммита номер rubizza-camp#79:

disable rubocop and delete empty lines

# Это сообщение коммита номер rubizza-camp#80:

disable rubocop anmd change config

# Это сообщение коммита номер rubizza-camp#81:

disable rubocop

# Это сообщение коммита номер rubizza-camp#82:

fix @values in dice_project

# Это сообщение коммита номер rubizza-camp#83:

try change commit

# Это сообщение коммита номер rubizza-camp#84:

fix error with changing

# Это сообщение коммита номер rubizza-camp#85:

disable rubocop in neo

# Это сообщение коммита номер rubizza-camp#86:

try fix errors with change commits

# Это сообщение коммита номер rubizza-camp#87:

delete superfluous string in strings

# Это сообщение коммита номер rubizza-camp#88:

For the second PR


# Это сообщение коммита номер rubizza-camp#89:

Add Reek-disable comments


# Это сообщение коммита номер rubizza-camp#90:

Fix locally Reek and Rubocop by adding comments


# Это сообщение коммита номер rubizza-camp#91:

Fix Regexp performance issue in extra_credit


# Это сообщение коммита номер rubizza-camp#92:

Add tests(neo.rb, rake, etc)


# Это сообщение коммита номер rubizza-camp#93:

Fix a problem with spelling and naming in commit messages


# Это сообщение коммита номер rubizza-camp#94:

Upload files from ruby_koans

# Это сообщение коммита номер rubizza-camp#95:

Solve about_exceptions

# Это сообщение коммита номер rubizza-camp#96:

Solve TriangleError

# Это сообщение коммита номер rubizza-camp#97:

Change negative? check to positive? in triangle

# Это сообщение коммита номер rubizza-camp#98:

Solve about iterations

# Это сообщение коммита номер rubizza-camp#99:

Solve about blocks

# Это сообщение коммита номер rubizza-camp#100:

Solve about sandwich code

# Это сообщение коммита номер rubizza-camp#101:

Solve scoring project

# Это сообщение коммита номер rubizza-camp#102:

Solve about classes

# Это сообщение коммита номер rubizza-camp#103:

Solve about open classes

# Это сообщение коммита номер rubizza-camp#104:

Solve dice project

# Это сообщение коммита номер rubizza-camp#105:

Solve about inheritance

# Это сообщение коммита номер rubizza-camp#106:

Solve about modules

# Это сообщение коммита номер rubizza-camp#107:

Solve about scope

# Это сообщение коммита номер rubizza-camp#108:

Solve about class methods

# Это сообщение коммита номер rubizza-camp#109:

Solve about message passing

# Это сообщение коммита номер rubizza-camp#110:

Solve about proxy object project

# Это сообщение коммита номер rubizza-camp#111:

Fix all koans

# Это сообщение коммита номер rubizza-camp#112:

Fix part rubocop complains

# Это сообщение коммита номер rubizza-camp#113:

resolve shown violation

# Это сообщение коммита номер rubizza-camp#114:

Fix second portion rubocop

# Это сообщение коммита номер rubizza-camp#115:

Fix all rubocop complains except triangle and scoring

# Это сообщение коммита номер rubizza-camp#116:

Fix sandwich code

# Это сообщение коммита номер rubizza-camp#117:

Rewrite scoring project

# Это сообщение коммита номер rubizza-camp#118:

Change triangle code

# Это сообщение коммита номер rubizza-camp#119:

Fix hound complains

# Это сообщение коммита номер rubizza-camp#120:

Fix configs

# Это сообщение коммита номер rubizza-camp#121:

Change configs back

# Это сообщение коммита номер rubizza-camp#122:

Change about_scoring for AbcSize

# Это сообщение коммита номер rubizza-camp#123:

Add and fix about_variable_scope

# Это сообщение коммита номер rubizza-camp#124:

Add newline in the end of path to enlightenment

# Это сообщение коммита номер rubizza-camp#125:

Change do-end

# Это сообщение коммита номер rubizza-camp#126:

Finish koans, fix hound

# Это сообщение коммита номер rubizza-camp#127:

Auto

# Это сообщение коммита номер rubizza-camp#128:

fix all koans

# Это сообщение коммита номер rubizza-camp#129:

3604 - 0 (rubizza-camp#120)

* Created folder with number

* Done all files, used rubocop and reek

* Update about_sandwich_code.rb,  error editing

* The second update about_sandwich_code, fixed error

* Fixed error Naming/ConstantName

* Fixed error Style/MutableConstant

* Fixed errors TooManyStatements and FeatureEnvy

* Fixed error Style/MutableConstant

* Updated file with new version code, used OOP

* Updated file

* Rewrote code, the code became clear

* Fixed syntax private methods and public

* Created normal name of method

# Это сообщение коммита номер rubizza-camp#130:

Initial commit

# Это сообщение коммита номер rubizza-camp#131:

fix rubocop

# Это сообщение коммита номер rubizza-camp#132:

update triangle.rb & change some rubocop

# Это сообщение коммита номер rubizza-camp#133:

Move to folder above 3548/0

# Это сообщение коммита номер rubizza-camp#134:

load tasks

fix about_asserts

fix about_nil

fix about_objects

fix about_arrays

fix about_array_assignment

fix about_hashes

fix about_strings

fix about_symbols

fix about_regular_expressions

fix about_methods

fix about_keyword_arguments

fix about_constants

fix about_control_statements

fix about_true_and_false

fix triangle

fix about_exceptions

fix about_iteration

fix about_blocks

fix about_sandwich_code

fix about_scoring_project

fix about_classes

fix about_open_classes

fix about_dice_project

fix about_inheritance

fix about_modules.rb

fix about_scope

fix about_class_methods

fix about_message_passing

fix about_proxy_object_project

fix about_to_str

fix about_variable_scope

fix code style according ruby code style

trying pass hound

fix rake result

pass hound

undo reek file

make new reek.yml

fix code style according reek

fix Layout/IndentationConsistency problem in about_scoring_project

rename param name in set_name method in about_classes; delete useless about_extra_credit; delete redundant comment in about_proxy_object_project; fix indentation in about_scoring_project; delete redundant line in method 'bark' in about_variable_scope

refactor method 'score' in about_scoring_project

another score refactoring

refactor method 'score': change conditions

# Это сообщение коммита номер rubizza-camp#135:

Done all, except extra task

# Это сообщение коммита номер rubizza-camp#136:

changed .hound.yml

# Это сообщение коммита номер rubizza-camp#137:

fix spacing

# Это сообщение коммита номер rubizza-camp#138:

solve reek problem

# Это сообщение коммита номер rubizza-camp#139:

argue with robocop

# Это сообщение коммита номер rubizza-camp#140:

make files in style
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hound hound[bot] hound[bot] left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@MargaretOm