Skip to content

Fix(workflows): don't run lint&format when editing a PR description - #677

Merged
Luhmos merged 1 commit into
mainfrom
fix/workflows/lintandformat-edited
Jun 16, 2026
Merged

Fix(workflows): don't run lint&format when editing a PR description#677
Luhmos merged 1 commit into
mainfrom
fix/workflows/lintandformat-edited

Conversation

@Marc-Andrieu

@Marc-Andrieu Marc-Andrieu commented May 7, 2026

Copy link
Copy Markdown
Member

Description

Summary

Just noticed that editing the description of #671 (which will happens a couple times) triggered lintandformat, and only it (the test workflows and others were already fixed somehow).

Changes Made

One-line change

Classification

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 🔨 Refactor (non-breaking change that neither fixes a bug nor adds a feature)
  • 🔧 Infra CI/CD (changes to configs of workflows)
  • 💥 BREAKING CHANGE (fix or feature that require a new minimal version of the front-end)
  • 😶‍🌫️ No impact for the end-users

Impact & Scope

  • Core functionality changes
  • Single module changes
  • Multiple modules changes
  • Other: workflows YAML

Testing

  • 1. Tested this locally
  • 2. Added/modified tests that pass the CI (or tested in a downstream fork)
  • 3. Tested in a local client using a pre-prod backend
  • 0. Untestable but trust me

Documentation

  • Updated the docs accordingly :
  • // Comments
  • No documentation needed

@Marc-Andrieu Marc-Andrieu self-assigned this May 7, 2026
@Marc-Andrieu Marc-Andrieu added ready for review This PR is ready to be reviewed fix bug This PR fix a bug workflow labels May 7, 2026
@codecov-commenter

codecov-commenter commented May 7, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 50.98%. Comparing base (6d502ea) to head (a2d801c).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #677   +/-   ##
=======================================
  Coverage   50.98%   50.98%           
=======================================
  Files         168      168           
  Lines        3789     3789           
=======================================
  Hits         1932     1932           
  Misses       1857     1857           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@NakoGH
NakoGH force-pushed the fix/workflows/lintandformat-edited branch from 09729ab to cd1bfc1 Compare May 27, 2026 09:34
@Luhmos
Luhmos force-pushed the fix/workflows/lintandformat-edited branch from cd1bfc1 to a2d801c Compare June 16, 2026 08:15
@Luhmos
Luhmos merged commit 9ac433b into main Jun 16, 2026
4 checks passed
@Luhmos
Luhmos deleted the fix/workflows/lintandformat-edited branch June 16, 2026 08:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix bug This PR fix a bug ready for review This PR is ready to be reviewed workflow

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants