Skip to content

Create helloworld.md#339

Open
ericsandu wants to merge 1 commit intorosedu:cdl-08from
ericsandu:helloworld
Open

Create helloworld.md#339
ericsandu wants to merge 1 commit intorosedu:cdl-08from
ericsandu:helloworld

Conversation

@ericsandu
Copy link
Copy Markdown

Built a helloworld.md file that complies 1/1 with the visual representation shown in helloworld-print.pdf

Signed-off-by: ericsandu <eric_andrei.sandu@stud.acs.upb.ro>
Copilot AI review requested due to automatic review settings March 22, 2026 11:29
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 a new helloworld.md Markdown document intended to reproduce the GitHub-rendered view captured in helloworld-print.pdf, showcasing “Hello, World!” examples across multiple languages.

Changes:

  • Introduces a new Markdown page with an image header, a language summary table, and per-language code blocks.
  • Adds build/run instructions for each language example (some currently placeholders).

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

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 singular (“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 build command is compiling helloworld.cpp, but the D example (and the repo’s sample file) is .d. This command won’t work as written; update it to compile the D source file name consistently.

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

Copilot uses AI. Check for mistakes.
Comment thread helloworld.md
Comment on lines +158 to +167
## x86_64 Assembly

```as
```

Build with:

```console
TODO
```
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 x86_64 Assembly section has an empty code block and TODO for the build command, so it doesn’t currently provide a runnable “Hello, World!” example. Either fill in the assembly source + toolchain commands or remove it from the summary table until it’s implemented.

Copilot uses AI. Check for mistakes.
Comment thread helloworld.md
Comment on lines +232 to +242
```php
<?php
echo "Hello, World!"
?>
```

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.

The PHP example as written won’t run: echo "Hello, World!" is missing a terminating semicolon, and the run command ./helloworld doesn’t invoke PHP (unless you add a shebang and make the script executable). Update the snippet and run instructions so they work together.

Copilot uses AI. Check for mistakes.
Comment thread helloworld.md
| Language | Language (Spec) Site | Section | Build / Run Toolchain | Debian / Ubuntu Packages |
|----------|----------------------|---------|-----------------------|--------------------------|
| C | [The Standard - C](https://www.iso-9899.info/wiki/The_Standard) | [C](#c) | GCC | `build-essential` |
| C++ | [The Standard - C++](https://isocpp.org/std/the-standard) | [C++](#c++) | GCC / G++ | `build-essential`, `g++` |
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 table link [C++](#c++) won’t resolve to the ## C++ header on GitHub (the generated header slug strips + and de-duplicates against ## C). Update the anchor to the actual generated slug for this header so the table link works.

Suggested change
| C++ | [The Standard - C++](https://isocpp.org/std/the-standard) | [C++](#c++) | GCC / G++ | `build-essential`, `g++` |
| C++ | [The Standard - C++](https://isocpp.org/std/the-standard) | [C++](#c-1) | GCC / G++ | `build-essential`, `g++` |

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

```rs
fn main() {
println!("Hello, World");
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 snippet prints "Hello, World" (missing !) even though the doc intro says programs print "Hello, World!". If Rust is intended to match the others, update the string to include the exclamation mark.

Suggested change
println!("Hello, World");
println!("Hello, World!");

Copilot uses AI. Check for mistakes.
Comment thread helloworld.md
Comment on lines +126 to +134
```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.

The Rust build/run commands are inconsistent: rustc hello.rs doesn’t match the section’s helloworld naming, and running ./helloworld won’t work unless the output binary is explicitly named that way. Make the source filename and output name consistent with the run command.

Copilot uses AI. Check for mistakes.
Comment thread helloworld.md
Comment on lines +177 to +186
## ARM64 Assembly

```as
```

Build with:

```console
TODO
```
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 ARM64 Assembly section is also empty and has TODO for the build command. As written it can’t be followed to produce output; please add the actual example + commands or drop it from the table for now.

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.

The Perl snippet is missing a trailing semicolon after the print(...) statement, which will cause a syntax error when run with perl. Add the semicolon so the example is valid.

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

Copilot uses AI. Check for mistakes.
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