Skip to content
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.

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

fix/ci #137

Merged
merged 2 commits into from
Jul 22, 2024
Merged

fix/ci #137

merged 2 commits into from
Jul 22, 2024

Conversation

Okm165
Copy link
Collaborator

@Okm165 Okm165 commented Jul 22, 2024

This part of the code includes test only when recursive layout is on. CI was running tests after configure step that failed in most cases because it changed layout and then tried to test resulting in test dependencies not found

@Okm165 Okm165 self-assigned this Jul 22, 2024
@Okm165 Okm165 requested review from fmkra and removed request for fmkra July 22, 2024 08:15
@Okm165 Okm165 added bug Something isn't working enhancement New feature or request labels Jul 22, 2024
@Okm165 Okm165 requested a review from fmkra July 22, 2024 08:19
@fmkra fmkra merged commit e5cb297 into main Jul 22, 2024
14 checks passed
@Okm165 Okm165 deleted the ci_fix branch July 22, 2024 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants