Skip to content

chore: remove skip ci#6950

Merged
OtavioStasiak merged 2 commits into
developfrom
chore.remove-skip-ci
Feb 4, 2026
Merged

chore: remove skip ci#6950
OtavioStasiak merged 2 commits into
developfrom
chore.remove-skip-ci

Conversation

@OtavioStasiak

@OtavioStasiak OtavioStasiak commented Jan 26, 2026

Copy link
Copy Markdown
Contributor

Proposed changes

Remove skip CI to enable run lint and merge PR's;

Issue(s)

https://rocketchat.atlassian.net/browse/CORE-1765

How to test or reproduce

Screenshots

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • Improvement (non-breaking change which improves a current function)
  • New feature (non-breaking change which adds functionality)
  • Documentation update (if none of the other choices apply)

Checklist

  • I have read the CONTRIBUTING doc
  • I have signed the CLA
  • Lint and unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works (if applicable)
  • I have added necessary documentation (if applicable)
  • Any dependent changes have been merged and published in downstream modules

Further comments

Summary by CodeRabbit

  • Chores
    • Updated CI workflow configuration to ensure continuous integration runs on automated commits.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai

coderabbitai Bot commented Jan 26, 2026

Copy link
Copy Markdown
Contributor

Walkthrough

The workflow configuration removes the [skip ci] suffix from a commit command in the prettier workflow, enabling CI checks to execute on prettier-related commits instead of bypassing them.

Changes

Cohort / File(s) Summary
Workflow Configuration
.github/workflows/prettier.yml
Removed [skip ci] suffix from commit command to allow CI execution on prettier commits

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • diegolmello

Poem

🐰 The skip flag takes its final bow,
As CI runs where admins allow,
Non-admins now can merge with cheer,
No special powers needed here! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'chore: remove skip ci' directly and concisely describes the main change in the pull request.
Linked Issues check ✅ Passed The PR removes [skip ci] from the workflow commit command, enabling CI runs for all PRs as required by CORE-1765.
Out of Scope Changes check ✅ Passed The changes are limited to the workflow file and directly address the objective of removing skip CI functionality with no unrelated modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore.remove-skip-ci

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@OtavioStasiak OtavioStasiak had a problem deploying to experimental_ios_build February 4, 2026 20:07 — with GitHub Actions Failure
@OtavioStasiak OtavioStasiak had a problem deploying to official_android_build February 4, 2026 20:07 — with GitHub Actions Failure
@OtavioStasiak OtavioStasiak had a problem deploying to experimental_android_build February 4, 2026 20:07 — with GitHub Actions Failure
@OtavioStasiak OtavioStasiak merged commit 5fa4072 into develop Feb 4, 2026
5 of 10 checks passed
@OtavioStasiak OtavioStasiak deleted the chore.remove-skip-ci branch February 4, 2026 20:07
diegolmello pushed a commit that referenced this pull request Feb 11, 2026
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.

2 participants