fix(web): resolve stylelint errors in bulk-messages page#902
Conversation
- Add empty line before rule as required by rule-empty-line-before - Use modern color-function notation rgb(0 0 0 / 4%) instead of rgba(0, 0, 0, 0.04) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Greptile SummaryThis PR fixes two stylelint rule violations in the
Confidence Score: 5/5This PR is safe to merge — it only touches two lines in a scoped CSS block with no runtime or functional impact. Both changes are purely stylistic linting corrections. The color values are semantically equivalent, and adding an empty line between CSS rules has no effect on rendering or behavior. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[CI Lint Check on main] -->|Stylelint errors detected| B[Failure]
B --> C[PR 902: Fix stylelint errors]
C --> D[Add empty line before hover rule]
C --> E[Convert to modern rgb notation]
D --> F[CI Lint Check passes]
E --> F
Reviews (1): Last reviewed commit: "fix(web): resolve stylelint errors in bu..." | Re-trigger Greptile |
- Split workflow into 'validate' (lint, test, build) and 'deploy' jobs - Validate runs on both pull_request and push to main - Deploy only runs after merge to main, gated behind validate passing Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| @@ -37,8 +40,26 @@ jobs: | |||
| - name: Run tests 🧪 | |||
| run: pnpm test | |||
|
|
|||
| - name: Debug 🐛 | |||
| run: echo GITHUB_SHA=${GITHUB_SHA} | |||
| - name: Build 🏗️ | |||
| run: mv .env.production .env && echo GITHUB_SHA=${GITHUB_SHA} >> .env && pnpm run generate | |||
|
|
|||
| deploy: | |||
| name: Deploy | |||
| needs: validate | |||
| if: github.event_name == 'push' && github.ref == 'refs/heads/main' | |||
| runs-on: ubuntu-latest | |||
|
|
|||
| steps: | |||
| - name: Checkout 🛎 | |||
| uses: actions/checkout@master | |||
|
|
|||
| - uses: pnpm/action-setup@v6 | |||
| name: Install pnpm | |||
| with: | |||
| version: 10 | |||
|
|
|||
| - name: Install dependencies 📦 | |||
| run: pnpm install | |||
|
|
|||
| - name: Build 🏗️ | |||
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 0 |
| Duplication | 0 |
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.
2 participants
Fixes the CI lint failure on main caused by stylelint errors in \pages/bulk-messages/index.vue:
ule-empty-line-before)
gba(0, 0, 0, 0.04)\ to modern notation
gb(0 0 0 / 4%)\ (\color-function-notation\ + \�lpha-value-notation)