Skip to content

fix(mcb): stabilize from clean origin/develop with conflict marker CI check - #216

Closed
marlon-costa-dc wants to merge 4 commits into
developfrom
fix/stable-mcb
Closed

fix(mcb): stabilize from clean origin/develop with conflict marker CI check#216
marlon-costa-dc wants to merge 4 commits into
developfrom
fix/stable-mcb

Conversation

@marlon-costa-dc

@marlon-costa-dc marlon-costa-dc commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator

Stabiliza mcb a partir de origin/develop clean.

Análise da situação:


Summary by cubic

Adds a CI workflow that fails pull requests to main when merge conflict markers are found in tracked files, preventing broken merges from reaching the default branch. The check ignores virtualenv, build, and vendor folders to avoid false positives.

Written for commit daeaff2. Summary will update on new commits.

Review in cubic

Test User and others added 3 commits August 9, 2026 20:15
@qodo-code-review

Copy link
Copy Markdown
Contributor

ⓘ Qodo reviews are paused because the subscription is no longer active. Ask your workspace admin to reactivate the subscription to resume reviews. Manage billing

@coderabbitai

coderabbitai Bot commented Aug 28, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 15 minutes.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 72167e14-d680-4d79-80dd-af6c23d4c33f

📥 Commits

Reviewing files that changed from the base of the PR and between b9d5c7f and daeaff2.

📒 Files selected for processing (1)
  • .github/workflows/conflict-marker-check.yml

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.

Comment on lines +9 to +23
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
with:
submodules: true
- name: Check for merge conflict markers
run: |
markers=$(git grep -l '<<<<<<< HEAD\|=======$' -- \
$(git ls-files | grep -vE '\.venv/|target/|dist/|\.cache/|_vendor/' | tr '\n' ' ') 2>/dev/null || true)
if [ -n "$markers" ]; then
echo "::error::Merge conflict markers found in tracked files:"
echo "$markers"
exit 1
fi
echo "No conflict markers found."
**Análise:**
- Repo no branch main (default)
- Broken merge PR #202 (69832e7) já revertido por c86edc2 no main
- 7 revert commits no histórico (reverts de código individual)
- Nenhum conflict marker no main
- CI: 8 workflows (7 existentes + 1 nova: conflict-marker-check)
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