Skip to content

Create my very own markdown file 😃 #340

Open
ericsandu wants to merge 4 commits into
rosedu:cdl-08from
ericsandu:myownmarkdown
Open

Create my very own markdown file 😃 #340
ericsandu wants to merge 4 commits into
rosedu:cdl-08from
ericsandu:myownmarkdown

Conversation

@ericsandu
Copy link
Copy Markdown

Adds a personally written markdown file containing all of the main structures as described in the workshop.
myownmarkdown.md contains a silly write up about my favorite food and showcases a very cool project related to it.

Signed-off-by: ericsandu <eric_andrei.sandu@stud.acs.upb.ro>
Signed-off-by: ericsandu <eric_andrei.sandu@stud.acs.upb.ro>
Copilot AI review requested due to automatic review settings March 22, 2026 11:57
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds new Markdown content to the workshop repository, including a personal “my favorite food” page and a multi-language Hello World reference page.

Changes:

  • Added myownmarkdown.md with headings, lists, tables, images, and an embedded Go code sample.
  • Added helloworld.md with a language/tooling summary table and Hello World snippets plus build/run instructions.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 9 comments.

File Description
myownmarkdown.md Personal markdown showcase (image, lists, table, and embedded Go snippet).
helloworld.md Hello World examples across languages with a summary table and run/build steps.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread helloworld.md
Run with:

```console
./helloworld
Copy link

Copilot AI Mar 22, 2026

Choose a reason for hiding this comment

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

PHP section run instructions are incorrect: ./helloworld won’t run a PHP script directly unless it’s executable with a shebang and correct filename. Use php helloworld.php (or make the script executable and document that).

Suggested change
./helloworld
php helloworld.php

Copilot uses AI. Check for mistakes.
Comment thread myownmarkdown.md
Comment on lines +1 to +3
# My favorite food

## Can you guess it?
Copy link

Copilot AI Mar 22, 2026

Choose a reason for hiding this comment

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

PR description mentions adding myownmarkdown.md, but this PR also introduces helloworld.md. Please update the PR description to reflect both files (or explain why helloworld.md is included).

Copilot uses AI. Check for mistakes.
Comment thread myownmarkdown.md Outdated
Comment thread myownmarkdown.md Outdated
Comment thread myownmarkdown.md
Comment on lines +41 to +45
Here is some cool Go code to do just that: [Original repository](https://github.com/harrybrwn/apizza)

```go
package cmd

Copy link

Copilot AI Mar 22, 2026

Choose a reason for hiding this comment

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

This embeds a large block of third-party code from an external repository. To avoid licensing/attribution issues, consider replacing it with a small excerpt, or add explicit license/attribution text consistent with that project’s license (a link alone may not be sufficient).

Copilot uses AI. Check for mistakes.
Comment thread helloworld.md
![helloworld](helloworld.png)

We list below Helloworld programs for different programming languages, i.e. programs that print "Hello, World!".
The specified compiler or interpreter is required for each programming languages.
Copy link

Copilot AI Mar 22, 2026

Choose a reason for hiding this comment

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

Grammar: “for each programming languages” should be “for each programming language”.

Suggested change
The specified compiler or interpreter is required for each programming languages.
The specified compiler or interpreter is required for each programming language.

Copilot uses AI. Check for mistakes.
Comment thread helloworld.md
Build with:

```console
gdc -Wall -o helloworld helloworld.cpp
Copy link

Copilot AI Mar 22, 2026

Choose a reason for hiding this comment

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

The D example’s build command compiles helloworld.cpp, but the snippet is D code. This should reference the D source file (e.g., helloworld.d) to avoid confusing/inaccurate instructions.

Suggested change
gdc -Wall -o helloworld helloworld.cpp
gdc -Wall -o helloworld helloworld.d

Copilot uses AI. Check for mistakes.
Comment thread helloworld.md
## Perl

```pl
print("Hello, World!\n")
Copy link

Copilot AI Mar 22, 2026

Choose a reason for hiding this comment

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

Perl snippet is missing a trailing semicolon, which makes it invalid Perl. Add the semicolon so the example runs as written.

Suggested change
print("Hello, World!\n")
print("Hello, World!\n");

Copilot uses AI. Check for mistakes.
Comment thread helloworld.md
Comment on lines +124 to +134
Build with:

```console
rustc hello.rs
```

Run with:

```console
./helloworld
```
Copy link

Copilot AI Mar 22, 2026

Choose a reason for hiding this comment

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

Rust section: rustc hello.rs will produce a hello binary by default, but the instructions say to run ./helloworld. Either specify -o helloworld (or rename the output) or update the run command to match the produced binary.

Copilot uses AI. Check for mistakes.
ericsandu and others added 2 commits March 22, 2026 15:28
Fix typo

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Fix typo

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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