pyvec / pyvec/naucse

Testování validity YAML souborů při buildu

Open
#17 5 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
2
Forks
11
PR merge metrics
No merged PRs in 30d

Description

Travis build 1173 na první pohled prošel, ale rozbil zobrazení kurzu, protože v info.yaml byly chybně pojmenované klíče. Chyba je vidět ve výstupu logu z arcy na konci buildu

Kouknul jsem na testy, ale nenašel jsem žádné nad reálnými daty.

Myslím si, že by bylo dobré build/deploy zastavit, pokud se v arce objeví chyby, aby se to potichu nerozbilo, i když se travis na první pohled tváří, že je vše v pořádku, protože testy funkcionality samotného naučse prošly bez problémů. Nejspíše to ale nepůjde hned, protože i následující build obsahuje v logu chyby s tímto nesouvisející, které by bylo třeba nejdříve opravit.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reviewing the Travis build 1173 log and the subsequent build log linked in the issue, then inspect how the real course data, including info.yaml, is processed during the build. Add coverage for invalid YAML or malformed keys and make the build/deploy outcome expose those errors without being masked by unrelated failures.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
build-system, ci-cd
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.