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

Remove mentions of org-ql-find-heading, org-ql-find-path from Changelog. #395

Merged
merged 4 commits into from
Jan 4, 2024
Merged

Remove mentions of org-ql-find-heading, org-ql-find-path from Changelog. #395

merged 4 commits into from
Jan 4, 2024

Conversation

ivanperez-keera
Copy link
Contributor

These commands were removed in 5768c68, but they were still mentioned in the release that was going to incorporate those two functions, v0.7.

@alphapapa
Copy link
Owner

Isn't org-ql-find-path extant?

@ivanperez-keera
Copy link
Contributor Author

I'm not sure what you mean.

According to 5768c68, org-ql-find-path was removed prior to the 0.7 release, and re-introduced in the 0.8 release (in commit ac2d435 specifically, which was only merged with the main branch in ace5e36.

As it pertains to the changelog, it should be removed from the 0.7 release subheading IMO.

As you prefer.

@ivanperez-keera
Copy link
Contributor Author

I've adjusted the commit message to make things a bit clearer. See if that helps and let me know if not.

alphapapa and others added 4 commits January 3, 2024 18:25
This fixes the case in which a user calls org-ql-find from a narrowed
buffer and chooses a result outside the restriction.  This is already
done in, e.g. org-ql-open-link; it's just an oversight here.
I don't fully understand why this is the correct fix, because I don't
know where the raw heading text is being linkified (it seems like the
:raw-value property should be the unlinkified text), but this seems to
work correctly now.  If it turns out to break something else, we'll
find out and fix it.

Fixes #282.

Reported-by: Jacob Boxerman <https://github.com/jakebox>
These commands were removed in 5768c68,
but they were still mentioned in the release that was going to
incorporate those two functions, v0.7.

This commit removes the mention of those two functions from the v0.7
subheading in the changelog. Note that `org-ql-find-path` was
re-introduced in v0.8, so it remains under that subheading as a separate
addition.
@alphapapa
Copy link
Owner

Ah, my mistake. Thanks.

@alphapapa alphapapa merged commit 02a2e25 into alphapapa:master Jan 4, 2024
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