Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added LINFO1104 exam solution from 2020 until 2022 + Exam solution for Projet P3 2023 Juin #900

Open
wants to merge 25 commits into
base: master
Choose a base branch
from

Conversation

Tfloow
Copy link
Contributor

@Tfloow Tfloow commented Jun 30, 2023

Norah and I made some better solution because the given one on the drive were not really great. Plus we used the latex doc format to make things looks more crisp.

  • I also added a github action to make sure that what ppl adds can be compiled (takes 3 minutes of runtime)
  • I changed the old logo of UCLouvain
  • I added the exam and its solution for 2023 Juin projet P3
  • Updated README with removing the forum epl which is no longer useful
  • Added the passing badge and the discord link to the server

@Tfloow Tfloow changed the title Added LINFO1104 exam solution from 2020 until 2022 Added LINFO1104 exam solution from 2020 until 2022 + Exam solution for Projet P3 2023 Juin Jul 1, 2023
@blegat
Copy link
Collaborator

blegat commented Jul 4, 2023

C'est peut-être plus facile de faire des PRs séparés pour des changements séparés et de ne pas travailler sur la branche master. Là, je pourrais déjà merger l'examen par exemple. Mais bon, le mieux est l'ennemi du bien donc pas besoin de changer ce PR ;)

| [<img src="https://cdn.pixabay.com/photo/2013/04/01/21/32/reading-99244_960_720.png" width="48">][doc-url] | [<img src="https://image.freepik.com/free-icon/pdf-file-format-symbol_318-45340.jpg" width="48">][pdf-url] | [![Join the chat at https://gitter.im/Gp2mv3/Syntheses](https://badges.gitter.im/Gp2mv3/Syntheses.svg)][chat-url] | [<img src="http://forum-epl.be/images/avatars/gallery/Photos/logo_phpBB.gif" width="96">][forum-url] | [<img src="https://gitforwindows.org/img/git_logo.png" width="48">][git-url]
| **Documentation** | **PDF** | **Chat** |**Git** | **Build** | **Discord**|
|:------------------:|:-------:|:--------:|:------:|:---------:|:----------:|
| [<img src="https://cdn.pixabay.com/photo/2013/04/01/21/32/reading-99244_960_720.png" width="48">][doc-url] | [<img src="https://image.freepik.com/free-icon/pdf-file-format-symbol_318-45340.jpg" width="48">][pdf-url] | [![Join the chat at https://gitter.im/Gp2mv3/Syntheses](https://badges.gitter.im/Gp2mv3/Syntheses.svg)][chat-url] | [<img src="https://gitforwindows.org/img/git_logo.png" width="48">][git-url] | [<img src="https://github.com/Tfloow/Syntheses/actions/workflows/main.yml/badge.svg" width="128">][build-url] | [<img src="https://upload.wikimedia.org/wikipedia/fr/thumb/4/4f/Discord_Logo_sans_texte.svg/106px-Discord_Logo_sans_texte.svg.png?20210527094128" width="64">][discord-url]
Copy link
Collaborator

Choose a reason for hiding this comment

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

On peut ajouter discord et mais garder le forum-epl. Même si c'est des années antérieures, on peut y retrouver des sujets qui sont encore utiles.

Copy link
Contributor

Choose a reason for hiding this comment

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

A priori le nom de domaine forum-epl.be est libre de ce que je vois.
image
Donc je suppose que c'est inutile de garder ce lien dans le README, à moins que quelqu'un ne le reprenne ?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Oui, je n'avais pas vu que le forum-epl était down :'( On peut le retirer du coup en effet

# Compile the document
ls
pdflatex "$part"
Done=true
Copy link
Collaborator

Choose a reason for hiding this comment

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

Pas sûr de ce qui est fait ici. est-ce que if !($Done) ne peut pas être remplacé par elif ?
Tu peux voir si le build a échoué avec $? ici

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.

3 participants