Skip to content

PRs trigger draft releases - #45

Open
tvolk131 wants to merge 3 commits into
masterfrom
pr_releases
Open

PRs trigger draft releases#45
tvolk131 wants to merge 3 commits into
masterfrom
pr_releases

Conversation

@tvolk131

@tvolk131 tvolk131 commented Dec 10, 2023

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • New Features

    • Implemented a new automated workflow for creating and publishing app releases.
  • Chores

    • Set up GitHub Actions to streamline release management.

@coderabbitai

coderabbitai Bot commented Dec 10, 2023

Copy link
Copy Markdown

Walkthrough

A new GitHub Actions workflow has been introduced to automate the process of creating and publishing a release. This workflow includes steps for setting up Node.js, extracting the application version, building a Tauri application, and finally, publishing the release. This streamlines the release process, ensuring consistency and efficiency.

Changes

File Change Summary
.github/workflows/pr-draft-release.yml New workflow added for automating the creation and publishing of releases, including Node.js setup, app version retrieval, Tauri app build, and release publication.

🐇✨
In the realm of code, a new path we weave,
Automating releases, on GitHub we achieve.
With Tauri's might and Node's gentle sway,
A rabbit crafts releases, hip-hip-hooray! 🚀🎉

Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on X ?


Tips

Chat with CodeRabbit Bot (@coderabbitai)

  • If you reply to a review comment from CodeRabbit, the bot will automatically respond.
  • To engage with CodeRabbit bot directly around the specific lines of code in the PR, mention @coderabbitai in your review comment
  • Note: Review comments are made on code diffs or files, not on the PR overview.
  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.

CodeRabbit Commands (invoked as PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger a review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai help to get help.

Note: For conversation with the bot, please use the review comments on code diffs or files.

CodeRabbit Configration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • The JSON schema for the configuration file is available here.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/coderabbit-overrides.v2.json

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review Status

Actionable comments generated: 3

Configuration used: CodeRabbit UI

Commits Files that changed from the base of the PR and between 66bafe6 and b7c96c0.
Files selected for processing (1)
  • .github/workflows/pr-draft-release.yml (1 hunks)
Additional comments: 3
.github/workflows/pr-draft-release.yml (3)
  • 19-20: Verify that the file path ./src-tauri/tauri.conf.json and the property package.version exist and are correct in the project structure.

The verification of the file path ./src-tauri/tauri.conf.json and the property package.version has been successful. The file exists in the project structure and contains the specified property with a valid version number.

  • 53-57: The job build-tauri installs dependencies only for Ubuntu. Ensure that platform-specific dependencies for macOS and Windows are also handled correctly.

  • 60-61: The Tauri action is used with a version tag of v0. Verify that this tag points to a stable and appropriate version for the project's needs.

@@ -0,0 +1,85 @@
name: "publish"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The workflow is named "publish," which could be misleading as it's intended for creating draft releases. Consider renaming it to something more descriptive, like "create-draft-release".

@@ -0,0 +1,85 @@
name: "publish"

on: pull_request

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The workflow is triggered on every pull request. Consider adding conditions to trigger this workflow only for specific branches or under certain conditions to avoid unnecessary runs.

Comment thread .github/workflows/pr-draft-release.yml Outdated
Comment on lines +72 to +84
- name: Publish release
id: publish-release
uses: actions/github-script@v6
env:
release_id: ${{ needs.create-release.outputs.release_id }}
with:
script: |
github.rest.repos.updateRelease({
owner: context.repo.owner,
repo: context.repo.repo,
release_id: process.env.release_id,
draft: false,
prerelease: false

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The publish-release job marks the release as no longer a draft without any checks. Consider adding steps to ensure that the build was successful and the release is ready to be published.

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.

1 participant