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

I112 bad para in wikilinks #114

Merged
merged 6 commits into from
Oct 1, 2022
Merged

Conversation

RobertDober
Copy link
Owner

No description provided.

RobertDober and others added 4 commits October 1, 2022 08:58
The line scanner was recognizing the | in a wikilink label as a table
column separator. This eventually still got turned into a link, but by
then, it was already wrapped in a Para block.

To fix this, eat anything wrapped in double square brackets before
looking for the table regex.

Co-authored-by: Ben Olive <sionide21@gmail.com>
@coveralls
Copy link

coveralls commented Oct 1, 2022

Pull Request Test Coverage Report for Build 96e7f6a2a3c4827c78e1323d93a68d390fa4aeb8-PR-114

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 18 of 18 (100.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build b1be87b8595bfe3f64d8b22e38c0dee63630975e: 0.0%
Covered Lines: 882
Relevant Lines: 882

💛 - Coveralls

@RobertDober RobertDober merged commit 11b422a into master Oct 1, 2022
@RobertDober RobertDober deleted the i112-bad-para-in-wikilinks branch October 1, 2022 14:23
RobertDober added a commit that referenced this pull request Oct 1, 2022
* Don't wrap labeled wikilinks in p tags (#113)

The line scanner was recognizing the | in a wikilink label as a table
column separator. This eventually still got turned into a link, but by
then, it was already wrapped in a Para block.

To fix this, eat anything wrapped in double square brackets before
looking for the table regex.

Co-authored-by: Ben Olive <sionide21@gmail.com>

* Added RELEASE NOTES for 1.4.28

* modernised wikilinks tests

* testing all: true implemented, bug for <sup> fixed

* fixing doctest for 1.11

* Prepare rel 1.4.28

Co-authored-by: Ben Olive <sionide21@gmail.com>
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.

None yet

2 participants