Skip to content

completed light cs section#101

Open
guy-car wants to merge 1 commit intofractal-nyc:mainfrom
guy-car:guillaume-caron/light-cs
Open

completed light cs section#101
guy-car wants to merge 1 commit intofractal-nyc:mainfrom
guy-car:guillaume-caron/light-cs

Conversation

@guy-car
Copy link
Copy Markdown

@guy-car guy-car commented May 30, 2025

Important

Adds guillaume.md with answers to basic computer science questions on complexity, pointers, data structures, and call stacks.

  • New File: Adds guillaume.md in computer-science/light-cs/answers/ with answers to basic CS questions.
  • Content:
    • Explains time complexity and space complexity using Big O notation.
    • Describes pointers and references, including their differences.
    • Defines data structures and lists types (linear, non-linear, primitive, non-primitive).
    • Explains the call stack and its importance in function execution and debugging.

This description was created by Ellipsis for 64e048b. You can customize this summary. It will automatically update as commits are pushed.

Copy link
Copy Markdown
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Important

Looks good to me! 👍

Reviewed everything up to 64e048b in 1 minute and 5 seconds. Click for details.
  • Reviewed 21 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. computer-science/light-cs/answers/guillaume.md:14
  • Draft comment:
    Add a newline at the end of the file for proper formatting.
  • Reason this comment was not posted:
    Decided after close inspection that this draft comment was likely wrong and/or not actionable: usefulness confidence = 10% vs. threshold = 50% While missing newlines at end of file can cause issues with some tools and is considered good practice, this is a very minor formatting issue that would likely be caught by linters or formatters. The comment doesn't point out any logical issues or code quality problems. It's more of a style/formatting suggestion. Missing newlines can cause real issues with some tools and version control systems. This could be considered a legitimate formatting problem worth fixing. While true, this is exactly the kind of minor formatting issue that should be handled by automated tools rather than manual review comments. It doesn't affect the actual content or logic. This comment should be removed as it's too minor of a formatting issue to warrant a manual review comment. This kind of thing should be handled by automated tooling.

Workflow ID: wflow_ecdd11Cp0OiC5SZI

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

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.

1 participant