Skip to content

spec & spec_answers: Change .to_not to .not_to - #79

Open
ticmyvom wants to merge 1 commit into
TheOdinProject:mainfrom
ticmyvom:not_to-consistency
Open

ticmyvom wants to merge 1 commit into
TheOdinProject:mainfrom
ticmyvom:not_to-consistency

Conversation

@ticmyvom

@ticmyvom ticmyvom commented Sep 14, 2026

Copy link
Copy Markdown

Address issue #59

Because

To follow the preference to use .not_to for negative expectations.

This PR

  • Change all .to_not to .not_to

Issue

Closes #59

Additional Information

Pull Request Requirements

  • I have thoroughly read and understand The Odin Project Contributing Guide
  • The title of this PR follows the location of change: brief description of change format, e.g. String spec: Update instructions for clarity
  • The Because section summarizes the reason for this PR
  • The This PR section has a bullet point list describing the changes in this PR
  • If this PR addresses an open issue, it is linked in the Issue section
  • If this PR includes changes in the spec folder, they are also updated in the corresponding file in the spec_answers folder (with passing tests).

Address issue TheOdinProject#59, to follow the preference to use .not_to for negative
expectations.
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.

.not_to/.to_not inconsistency

1 participant