Skip to content

feat(workflows): add gh release instead of ncipollo/release-action#1392

Open
dzhalaevd wants to merge 3 commits intopytest-dev:mainfrom
dzhalaevd:1382-remove-ncipollo-deps
Open

feat(workflows): add gh release instead of ncipollo/release-action#1392
dzhalaevd wants to merge 3 commits intopytest-dev:mainfrom
dzhalaevd:1382-remove-ncipollo-deps

Conversation

@dzhalaevd
Copy link
Copy Markdown

@dzhalaevd dzhalaevd commented Mar 30, 2026

What was wrong?

Due to security issues with ncipollo/release-action need to remove this dependency and replace with gh release

Closes: #1382
Related: #1382

How it was fixed?

The release logic previously implemented via ncipollo/release-action has been reimplemented using the gh

Our workflow now relied on two key flags:

  • allowUpdates: true
  • skipIfReleaseExists: true

These flags overlap in behavior at least semantically and the ncipollo documentation doesn't clearly define their interaction or precedence. Based on the existing workflow, i was reimplemented as follows with this idea:

if release exists:
    if draft:
        gh release edit
    if published:
        exit
else:
    gh release create

I tried run jobs for check how release creating here
and re-run for checking how it was updates

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.13%. Comparing base (b6f574c) to head (0dcce49).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1392   +/-   ##
=======================================
  Coverage   95.13%   95.13%           
=======================================
  Files           2        2           
  Lines         473      473           
  Branches       57       57           
=======================================
  Hits          450      450           
  Misses         17       17           
  Partials        6        6           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dzhalaevd dzhalaevd changed the title [WIP] feat(workflows): add gh release instead of ncipollo/release-action feat(workflows): add gh release instead of ncipollo/release-action Apr 1, 2026
@dzhalaevd dzhalaevd marked this pull request as ready for review April 1, 2026 08:31
Copy link
Copy Markdown
Author

@dzhalaevd dzhalaevd left a comment

Choose a reason for hiding this comment

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

I think this is ready for review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove dependency on ncipollo/release-action

2 participants