Skip to content
This repository has been archived by the owner on May 29, 2024. It is now read-only.

Uphold okp4 actions #59

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Uphold okp4 actions #59

wants to merge 1 commit into from

Conversation

NourKired
Copy link
Contributor

No description provided.

@NourKired NourKired force-pushed the ci/uphold-okp4-workflows branch from 45f357b to b76a4e9 Compare December 28, 2022 14:08
@NourKired NourKired force-pushed the ci/uphold-okp4-workflows branch from b76a4e9 to 1eb6b0c Compare December 28, 2022 14:12
@NourKired NourKired requested a review from ad2ien December 28, 2022 14:18
Copy link
Contributor

@ad2ien ad2ien left a comment

Choose a reason for hiding this comment

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

Hi, there's something weird with publish package step. And it could be called something like update ci actions 😉

@@ -99,10 +99,16 @@ jobs:
with:
poetry-version: 1.3.1

- name: Publish package
Copy link
Contributor

Choose a reason for hiding this comment

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

there's some code duplication around here 😉

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants