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 hs.spaces so it works when hs.spaces.screensHaveSeparateSpaces() is false #3196

Merged
merged 3 commits into from
Jul 1, 2022

Conversation

asmagill
Copy link
Member

This still needs to be tested when a second monitor is attached and then I need to run through my tests again switching "Screens Have Separate Spaces" back on to make sure I didn't break anything.

Should be able to finalize tonight/tomorrow.

@asmagill asmagill added bug enhancement pr-fix Pull Request implementing a bug fix labels Apr 23, 2022
@github-actions
Copy link

github-actions bot commented Apr 23, 2022

View Test Results

    1 files      2 suites   11m 13s ⏱️
342 tests 289 ✔️ 50 💤 3

For more details on these failures, see this check.

Results for commit d657566.

♻️ This comment has been updated with latest results.

@asmagill asmagill changed the title [WIP] Fix hs.spaces so it works when hs.spaces.screensHaveSeparateSpaces() is false [READY] Fix hs.spaces so it works when hs.spaces.screensHaveSeparateSpaces() is false Apr 24, 2022
@asmagill
Copy link
Member Author

I hate CI

As far as I can tell the errors are unrelated, so this is ready to go unless anyone has objections.

@latenitefilms
Copy link
Contributor

Yeah, it looks like hs.serial is now that GitHub Actions is running on Monterey hardware - doh!

@latenitefilms
Copy link
Contributor

I've pushed to ignore these two tests in #3198

@cmsj cmsj merged commit 4634b81 into Hammerspoon:master Jul 1, 2022
@cmsj cmsj changed the title [READY] Fix hs.spaces so it works when hs.spaces.screensHaveSeparateSpaces() is false Fix hs.spaces so it works when hs.spaces.screensHaveSeparateSpaces() is false Mar 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug enhancement pr-fix Pull Request implementing a bug fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants