Skip to content

Update CI#619

Closed
antisaling wants to merge 1 commit into
kworkflow:unstablefrom
antisaling:update-ci
Closed

Update CI#619
antisaling wants to merge 1 commit into
kworkflow:unstablefrom
antisaling:update-ci

Conversation

@antisaling
Copy link
Copy Markdown
Contributor

This commit updates CI hooks and separates tasks as logical steps to
facilitate overviewing failing jobs.

Signed-off-by: Isabella Basso isabbasso@riseup.net

@antisaling
Copy link
Copy Markdown
Contributor Author

@rodrigosiqueira @kwy95 I'd appreciate some help on this one, as I'm not sure what's missing here.

Comment thread .github/workflows/test_setup_and_docs.yml Outdated
Comment thread .github/workflows/unit_tests.yml
@rodrigosiqueira
Copy link
Copy Markdown
Collaborator

@isinyaaa don't forget to rebase this PR.

@antisaling antisaling force-pushed the update-ci branch 2 times, most recently from 64e8a2a to 8fa7049 Compare May 27, 2022 18:19
@antisaling
Copy link
Copy Markdown
Contributor Author

antisaling commented May 27, 2022

I'm still unsure about what's causing kcov to fail... There were many things installed during the setup which kcov might be using, though I don't know either how it works or how to go about reading its output.

@rodrigosiqueira
Copy link
Copy Markdown
Collaborator

@kwy95 any idea?

Copy link
Copy Markdown
Collaborator

@rodrigosiqueira rodrigosiqueira left a comment

Choose a reason for hiding this comment

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

Hi @isinyaaa I added some ideas, could you try them?

Comment thread .github/workflows/kcov.yml
Comment thread .github/workflows/kcov.yml Outdated
@antisaling antisaling force-pushed the update-ci branch 8 times, most recently from 6abcb7e to 9da53e1 Compare June 7, 2022 16:49
This commit updates CI hooks and separates tasks as logical steps to
facilitate overviewing failing jobs.

Signed-off-by: Isabella Basso <isabbasso@riseup.net>
@antisaling
Copy link
Copy Markdown
Contributor Author

DONE!

@rodrigosiqueira
Copy link
Copy Markdown
Collaborator

Veryyyyy nice PR! Merged to the unstable branch :)
Thanks a lot.

1 similar comment
@rodrigosiqueira
Copy link
Copy Markdown
Collaborator

Veryyyyy nice PR! Merged to the unstable branch :)
Thanks a lot.

@antisaling antisaling deleted the update-ci branch June 12, 2022 14:57
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