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

Exercise 2.77 and refactor parts of SICP chapter 2's code #166

Merged
merged 4 commits into from
Feb 18, 2024
Merged

Conversation

SmetDenis
Copy link
Owner

Also,

Refactor parts of SICP chapter 2's code

The code refactor focused on the parts related to packages in chapter 2 of SICP. It included separating different packages into their own files, implementing a generic operation system, and standardizing the function interface. This has resulted in cleaner, more maintainable, and more understandable code.

Denis Smet added 4 commits February 4, 2024 23:39
The number of completed exercises was updated in the progress bar of the README, demonstrating more tasks have been accomplished. Additionally, code related to exercise 2.77 has been added, which covers generic arithmetic operations.
# Conflicts:
#	README.md
The code refactor focused on the parts related to packages in chapter 2 of SICP. It included separating different packages into their own files, implementing a generic operation system, and standardizing the function interface. This has resulted in cleaner, more maintainable, and more understandable code.
@SmetDenis SmetDenis changed the title Exercise 2.77 Exercise 2.77 and refactor parts of SICP chapter 2's code Feb 18, 2024
@coveralls
Copy link

Pull Request Test Coverage Report for Build 7950737312

Details

  • 95 of 124 (76.61%) changed or added relevant lines in 9 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.3%) to 93.163%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/sicp/chapter_2/part_4/book_2_4.clj 4 6 66.67%
src/sicp/chapter_2/packages/install_rectangular.clj 11 16 68.75%
src/sicp/chapter_2/packages/op_table.clj 16 21 76.19%
src/sicp/chapter_2/packages/install_polar.clj 9 16 56.25%
src/sicp/chapter_2/packages/install_complex.clj 13 23 56.52%
Totals Coverage Status
Change from base Build 7864893950: -0.3%
Covered Lines: 2330
Relevant Lines: 2501

💛 - Coveralls

@SmetDenis SmetDenis merged commit 550f8d7 into main Feb 18, 2024
7 checks passed
@SmetDenis SmetDenis deleted the 277 branch February 18, 2024 18:03
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.

2 participants