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

Organize tests and start Hyprland testing #3345

Merged
merged 13 commits into from
Jun 10, 2024
Merged

Conversation

khaneliman
Copy link
Contributor

@khaneliman khaneliman commented Jun 9, 2024

Sorry if this is too all over the place for a single PR... just kinda throwing various things together as I came across them and just wanted to fix them...

  • Organized the test folder a bit so that its easier to see which test suites are failing and manage them as they scale in size.
  • Start hyprland tests, just testing the getSocketFolder change recently.
    • This test suite caught a possible crash if someone unsets the XDG_RUNTIME_DIR before launching. Now, we continue to fallback to /tmp/hypr without seg fault.
  • Minor clang-format cleanup that was failing CI
  • Change linux builders to not cancel each other if one fails
  • Add python3-packaging dependency for opensuse image
    • Tested building docker image locally and then building in the opensuse container, fixed build.
    • NOTE: CI will still fail until an image is pushed with the fix.
  • Bump spdlog to next minor version

Fixes alpine build and is a commonly distributed version
@khaneliman
Copy link
Contributor Author

khaneliman commented Jun 9, 2024

@Alexays Did you need me to break this up or will this PR be fine in this state? Some of the changes are just fixing the CI checks.

@Alexays
Copy link
Owner

Alexays commented Jun 10, 2024

LGTM, thx!

@Alexays Alexays merged commit 0251e25 into Alexays:master Jun 10, 2024
8 of 9 checks passed
@khaneliman khaneliman deleted the testing branch June 10, 2024 19:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants