Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: add PR title checker and release workflow #1705

Merged
merged 1 commit into from
Jan 22, 2025

Conversation

Junjiequan
Copy link
Member

@Junjiequan Junjiequan commented Jan 20, 2025

Description

This PR includs a workflow to automize release process

Tests included

  • Included for each change/fix?
  • Passing? (Merge will not be approved unless this is checked)

Documentation

  • swagger documentation updated [required]
  • official documentation updated [nice-to-have]

official documentation info

If you have updated the official documentation, please provide PR # and URL of the pages where the updates are included

Backend version

  • Does it require a specific version of the backend
  • which version of the backend is required:

Summary by Sourcery

Add workflows for release bumping and PR title checking.

CI:

  • Add a CI workflow to bump the release version and build/push the Docker image when a commit is pushed to the release branch.
  • Add a CI workflow to check the title of pull requests targeting the master branch.

Summary by Sourcery

Add CI workflows to automate release bumping and PR title checking.

CI:

  • Add a workflow to bump the release version, build, and push the Docker image when a commit is pushed to the release branch.
  • Add a workflow to check the title of pull requests targeting the master branch.

@Junjiequan Junjiequan marked this pull request as ready for review January 20, 2025 10:20
Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey @Junjiequan - I've reviewed your changes - here's some feedback:

Overall Comments:

  • The PR title checker configuration file (.github/pr-title-checker-config.json) is referenced but not included in the changes. Please add the configuration file with the desired PR title format rules.
  • Please complete the checklist items in the PR description, particularly regarding testing confirmation and documentation updates.
Here's what I looked at during the review
  • 🟡 General issues: 1 issue found
  • 🟢 Security: all looks good
  • 🟢 Testing: all looks good
  • 🟢 Complexity: all looks good
  • 🟢 Documentation: all looks good

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

.github/workflows/release-push.yml Show resolved Hide resolved
Copy link
Collaborator

@martin-trajanovski martin-trajanovski left a comment

Choose a reason for hiding this comment

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

Looks good to me

@Junjiequan Junjiequan force-pushed the automate-FE-release-process branch from 73f0e15 to 99ae522 Compare January 22, 2025 09:15
@Junjiequan Junjiequan enabled auto-merge (squash) January 22, 2025 09:15
@Junjiequan Junjiequan merged commit 903a5b3 into master Jan 22, 2025
7 checks passed
@Junjiequan Junjiequan deleted the automate-FE-release-process branch January 22, 2025 09:22
Junjiequan added a commit that referenced this pull request Jan 22, 2025
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.

3 participants