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

Remove PR message workflow check for bots #2598

Merged
merged 1 commit into from
Feb 12, 2025

Conversation

Incogdino
Copy link
Contributor

@Incogdino Incogdino commented Feb 9, 2025

Previously, adding a new contributor triggers a PR message workflow to check for PR messages. By default, no PR message is added by the bot.

Removing the check for PR messages for bots will likely not affect any essential checks.

What is the purpose of this pull request?

  • Documentation update
  • Bug fix
  • [] Feature addition or enhancement
  • Code maintenance
  • DevOps
  • Improve developer experience
  • Others, please explain:

Closes #2561

Overview of changes:
Removed pr message workflow check for all bots.

Anything you'd like to highlight/discuss:
The updated workflow to check for PR message will skip for all bots.

Testing instructions:
image

The CI test was cloned and tested on a private GitHub repo.

Proposed commit message: (wrap lines at 72 characters)
Remove PR message workflow check for bots

Previously, adding a new contributor triggers a PR message workflow
to check for PR messages. By default, no PR message is added
by the bot.

Removing the check for PR messages for bots will likely not affect
any essential checks.


Checklist: ☑️

  • Updated the documentation for feature additions and enhancements
  • Added tests for bug fixes or features
  • Linked all related issues
  • No unrelated changes

Reviewer checklist:

Indicate the SEMVER impact of the PR:

  • Major (when you make incompatible API changes)
  • Minor (when you add functionality in a backward compatible manner)
  • Patch (when you make backward compatible bug fixes)

At the end of the review, please label the PR with the appropriate label: r.Major, r.Minor, r.Patch.

Breaking change release note preparation (if applicable):

  • To be included in the release note for any feature that is made obsolete/breaking

Give a brief explanation note about:

  • what was the old feature that was made obsolete
  • any replacement feature (if any), and
  • how the author should modify his website to migrate from the old feature to the replacement feature (if possible).

Previously, adding a new contributor triggers a PR message workflow to
check for PR messages. By default, no PR message is added by the bot.

Removing the check for PR messages for bots will likely not affect any
essential checks.
Copy link

codecov bot commented Feb 9, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 51.86%. Comparing base (4ed8781) to head (0a52447).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2598   +/-   ##
=======================================
  Coverage   51.86%   51.86%           
=======================================
  Files         127      127           
  Lines        5474     5474           
  Branches     1201     1201           
=======================================
  Hits         2839     2839           
  Misses       2340     2340           
  Partials      295      295           

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

Copy link
Contributor

@tlylt tlylt left a comment

Choose a reason for hiding this comment

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

👍

@tlylt tlylt added the r.Patch Version resolver: increment by 0.0.1 label Feb 12, 2025
@tlylt tlylt merged commit fc2893d into MarkBind:master Feb 12, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
r.Patch Version resolver: increment by 0.0.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Skip PR message reminder check for all-contributors bot
2 participants