Skip to content

Conversation

akirak
Copy link
Contributor

@akirak akirak commented Jun 13, 2021

This is a set of tests to ensure #220 is fixed.

These tests should fail without the changes from the PR and should succeed after the PR is merged.

@akirak
Copy link
Contributor Author

akirak commented Jun 13, 2021

@alphapapa I've added a new Org entry to the test data, which breaks some existing tests. Could you handle it? I think you have to add another link to the document and add some :regexp cases. I don't mind if you work on it elsewhere and close this PR.

@akirak akirak force-pushed the extra-link-tests branch from 414eadc to 216e01a Compare June 13, 2021 16:43
@akirak akirak force-pushed the extra-link-tests branch from 216e01a to 0392b53 Compare June 13, 2021 16:48
@akirak akirak force-pushed the extra-link-tests branch from 0392b53 to 9556173 Compare June 13, 2021 16:50
@alphapapa alphapapa self-assigned this Jun 16, 2021
@alphapapa alphapapa added this to the 0.5.2 milestone Jun 16, 2021
@alphapapa
Copy link
Owner

Thanks. I'd prefer not to modify the test data unless absolutely necessary, so I'll see...

@alphapapa alphapapa added the bug label Jun 16, 2021
@alphapapa alphapapa closed this in c57f097 Jun 16, 2021
US-Thakur pushed a commit to US-Thakur/merge_testing that referenced this pull request Mar 14, 2022
Closes alphapapa/org-ql#222.  See also
alphapapa/org-ql#220.

Thanks to Akira Komamura (@akirak).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants