Skip to content

change -2 to -3 to align with the following graph and text AND remove duplicate word determine#11

Open
ascendho wants to merge 2 commits into
cs106l:mainfrom
ascendho:main
Open

change -2 to -3 to align with the following graph and text AND remove duplicate word determine#11
ascendho wants to merge 2 commits into
cs106l:mainfrom
ascendho:main

Conversation

@ascendho
Copy link
Copy Markdown

Fix issue I opened earlier:

The issue reports a discrepancy in the “Sequence Containers/Behind the Scenes” section of the CS106L textbook: the code snippet shows pushing -2 to the front of deque d, but the accompanying graph and explanatory text refer to pushing -3. The value used in the code does not match the value shown in the graph and text, and they should be made consistent.

Summary generated by Copilot:

This pull request makes a minor change to the sequence of values being pushed to the front of a container in the src/05-containers/01-sequence-containers.md file. The value -2 is replaced with -3 in the example.

@ascendho
Copy link
Copy Markdown
Author

Fix issue I opened earlier:

The issue reports a discrepancy in the “Sequence Containers/Behind the Scenes” section of the CS106L textbook: the code snippet shows pushing -2 to the front of deque d, but the accompanying graph and explanatory text refer to pushing -3. The value used in the code does not match the value shown in the graph and text, and they should be made consistent.

Summary generated by Copilot:

This pull request makes a minor change to the sequence of values being pushed to the front of a container in the src/05-containers/01-sequence-containers.md file. The value -2 is replaced with -3 in the example.

Another issue I submitted.

@ascendho ascendho changed the title change -2 to -3 to align with the following graph and text change -2 to -3 to align with the following graph and text AND remove duplicate word determine Sep 18, 2025
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.

code element unmatched to graph and text

1 participant