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 OrgPromise:next() being triggered before original promise is finished #29

Merged
merged 7 commits into from
Apr 27, 2024

Conversation

chipsenkbeil
Copy link
Owner

No description provided.

@chipsenkbeil chipsenkbeil changed the title print Fix OrgPromise:next() being triggered before original promise is finished Apr 27, 2024
@chipsenkbeil
Copy link
Owner Author

@kristijanhusak I'm not sure why, but for neovim nightly my test kept failing where I create a window and (optionally) switch focus to it. The solution was to remove the promise chaining I was doing, and instead inline the call to set the focused window.

This doesn't make sense to me, but figured I'd highlight it in case there's something weird about neovim nightly that's messing with the promise library in orgmode.

@chipsenkbeil chipsenkbeil merged commit 34a9468 into main Apr 27, 2024
4 checks passed
@chipsenkbeil chipsenkbeil deleted the fix/FixTestsFailingOnNightly branch April 27, 2024 02:30
@chipsenkbeil chipsenkbeil mentioned this pull request Apr 27, 2024
chipsenkbeil added a commit that referenced this pull request May 6, 2024
…shed (#29)

* Fix focusing of roam buffer by inlining the call
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.

1 participant