Skip to content

chore: make cyclic deps check an error #38543

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
mohanarpit merged 25 commits into from
Jan 9, 2025
Merged

chore: make cyclic deps check an error #38543

Changes from 1 commit
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
086bd6e
chore: make cyclic deps check an error
Jan 8, 2025
370b0e5
add client-check-cyclic-deps as mandatory check for qc-result
Jan 8, 2025
903a8e4
add test cyclic dependency
Jan 8, 2025
fc56f5a
add result check for client-check-cyclic-deps
Jan 8, 2025
d774508
update emoji for Cyclic Dependency Check
Jan 8, 2025
fecf2d4
Update command to count and add comments
Jan 8, 2025
086cc7a
fix comment typo
Jan 8, 2025
116dc9b
Revert "add test cyclic dependency"
Jan 8, 2025
12c0947
install dpdx globally
Jan 8, 2025
b09518e
test changes
Jan 8, 2025
6ea01fa
Revert "test changes"
Jan 8, 2025
ef115da
add file extensions to dpdm glob pattern
Jan 9, 2025
4acb4e2
Add same error emojis as cypress test failure
Jan 9, 2025
8aa14a5
fix typo in step id
Jan 9, 2025
846eed4
add quotes to shell script variables to prevent word splitting
Jan 9, 2025
f4a948b
fix typo in variable
Jan 9, 2025
7d4dece
wrap all shell variables with quotes
Jan 9, 2025
c053aeb
trigger cyclic deps check
Jan 9, 2025
e1e0742
revert: trigger cyclic deps check
Jan 9, 2025
658e605
yarn before cyclic dependency check
Jan 9, 2025
66fd9f8
move install deps to a seperate step and load cached node_modules
Jan 9, 2025
b993677
revert: trigger cyclic deps check
Jan 9, 2025
b4e9182
remove restore cache steps
Jan 9, 2025
57e0bd3
test changes
Jan 9, 2025
49cca4c
revert: trigger cyclic deps check
Jan 9, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .github/workflows/ci-client-cyclic-deps-check.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -85,3 +85,8 @@ jobs:
issue_number: prNumber,
body: message
});

# Fail the workflow if cyclic dependencies are found
- name: Fail the workflow if cyclic dependencies are found
if: steps.compare-deps.outputs.has_more_cyclic_deps == 'true' && steps.changed-files.outputs.any_changed == 'true'
run: exit 1