fix: kroki.io broken service prevents ci from passing #200

stevenj · 2024-03-08T09:50:24Z

Description

Kroki is breaking CI
This PR disables Kroki altogether and enables a replacement for rendering graphviz documents.

Please confirm the following checks

My code follows the style guidelines of this project
I have performed a self-review of my code
I have commented my code, particularly in hard-to-understand areas
I have made corresponding changes to the documentation
My changes generate no new warnings
I have added tests that prove my fix is effective or that my feature works
New and existing unit tests pass locally with my changes
Any dependent changes have been merged and published in downstream module

Closes #202

…put-hk/catalyst-ci into feat/add-python-linter

…in rust

earthly/rust/Earthfile

.config/dictionaries/project.dic

Co-authored-by: Alex Pozhylenkov <[email protected]>

…put-hk/catalyst-ci into fix/kroki-broken-with-graphviz

FelipeRosa

LGTM

alicechaitea and others added 30 commits February 22, 2024 10:47

feat: add python linter

7b4e5b6

fix: combine RUN at line 10 after debugging

59e55e6

fix: update Earthfile with example project

2492e67

fix: fixes pillow (9.5.0) not supporting PEP 517 builds

2f74af4

fix: updates pillow to version 10.2.0

cb508e2

fix: addresses pillow version outdated issue

ec1d99e

fix: updates pillow version

4f0e82f

Merge branch 'master' into feat/add-python-linter

7219d95

fix downgrades pillow to 10.0.0

b767697

Merge branch 'feat/add-python-linter' of https://github.com/input-out…

1a0d8ee

…put-hk/catalyst-ci into feat/add-python-linter

fix: downgrades pillow back to 9.5.0

db159f7

fix: downgrades pillow back to 9.5.0

5b81d2c

test: pep517 error

b3cb040

fix: adds zlib to Earthfile

e6bca0f

fix: adds the word FUNCTION to Earthfile

335e56b

fix: addresses parser error

1ef0d1f

fix: address parser issue

b826216

fix: addresses parser error

f26d37a

fix: address parser issue

c8a6ca3

fix: address parser issue

84332a8

fix: address parser issue

97f75f7

fix: address parser issue

eb8a45d

fix: addresses directory issue

2d1018f

fix: adds image processing libraries

fa3b837

fix: adds image processing libraries

6697ff3

fix: adds image processing libraries

d037145

fix: updates depedendencies

63e0a43

fix: updates depedendencies

1c4a022

fix: adds --no-root flag when installing poetry

298fe85

fix: changes python linter to ruff

a7f20a9

alicechaitea and others added 16 commits February 29, 2024 16:25

fix: addresses build.sh not found

ef06493

fix: addresses build.sh not found

946f3dd

fix: addresses /scripts not found

df96533

fix: addresses build.sh not found

10d4888

fix: addresses build.sh not found

25c3699

fix: addresses build.sh not found

cae22ff

fix: addresses build.sh not found

dbd5784

fix: adds build.sh to utilities/scripts/bash

3ade995

fix: deletes duplicate build.sh

00581a7

fix: changes dir for build.sh

1231e6f

fix: adds dir for build.sh

658149f

fix: deletes dir

1e0ce06

docs(docs): Fix docs earthfiles

a6ba221

fix(docs): Fix docs building issue

375b36a

Merge branch 'master' into fix/kroki-broken-with-graphviz

97c2626

fix(docs): Disable Kroki because its breaking CI

be1cfc3

stevenj enabled auto-merge (squash) March 8, 2024 09:51

stevenj added the bug Something isn't working label Mar 8, 2024

stevenj self-assigned this Mar 8, 2024

fix(rust): disable global cahce and see if it fixes ci build failure …

0dd2016

…in rust

Mr-Leshiy reviewed Mar 8, 2024

View reviewed changes

earthly/rust/Earthfile Outdated Show resolved Hide resolved

Mr-Leshiy reviewed Mar 8, 2024

View reviewed changes

.config/dictionaries/project.dic Outdated Show resolved Hide resolved

stevenj and others added 4 commits March 8, 2024 18:43

fix(rust): change didn't help CI so revert it

92a1135

Update .config/dictionaries/project.dic

95658f0

Co-authored-by: Alex Pozhylenkov <[email protected]>

fix(cspell): spelling correction

ba2bc84

Merge branch 'fix/kroki-broken-with-graphviz' of github.com:input-out…

08611c0

…put-hk/catalyst-ci into fix/kroki-broken-with-graphviz

stevenj requested a review from Mr-Leshiy March 8, 2024 11:51

FelipeRosa approved these changes Mar 8, 2024

View reviewed changes

stevenj merged commit 7ede49e into master Mar 8, 2024
65 checks passed

stevenj deleted the fix/kroki-broken-with-graphviz branch March 8, 2024 11:55

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: kroki.io broken service prevents ci from passing #200

fix: kroki.io broken service prevents ci from passing #200

stevenj commented Mar 8, 2024 •

edited

Loading

FelipeRosa left a comment

fix: kroki.io broken service prevents ci from passing #200

fix: kroki.io broken service prevents ci from passing #200

Conversation

stevenj commented Mar 8, 2024 • edited Loading

Description

Please confirm the following checks

FelipeRosa left a comment

Choose a reason for hiding this comment

stevenj commented Mar 8, 2024 •

edited

Loading