[PR #25900] chore(workflows): remove redundant eslint command from style workflow #31255

Closed
opened 2026-02-21 20:49:07 -05:00 by yindo · 0 comments
Owner

Original Pull Request: https://github.com/langgenius/dify/pull/25900

State: closed
Merged: Yes


Summary

  • Removed duplicate 'pnpm run eslint' command from GitHub style workflow
  • 'pnpm run lint' already includes ESLint checking, making the separate command redundant

Changes

  • Simplified .github/workflows/style.yml by removing unnecessary eslint step

Benefits

  • Reduces CI execution time by eliminating redundant linting
  • Streamlines workflow configuration
  • Maintains same linting coverage since 'pnpm run lint' includes ESLint
**Original Pull Request:** https://github.com/langgenius/dify/pull/25900 **State:** closed **Merged:** Yes --- ## Summary - Removed duplicate 'pnpm run eslint' command from GitHub style workflow - 'pnpm run lint' already includes ESLint checking, making the separate command redundant ## Changes - Simplified .github/workflows/style.yml by removing unnecessary eslint step ## Benefits - Reduces CI execution time by eliminating redundant linting - Streamlines workflow configuration - Maintains same linting coverage since 'pnpm run lint' includes ESLint
yindo added the pull-request label 2026-02-21 20:49:07 -05:00
yindo closed this issue 2026-02-21 20:49:07 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/dify#31255