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

Create new parser system #148

Merged
merged 22 commits into from
Feb 6, 2023
Merged

Create new parser system #148

merged 22 commits into from
Feb 6, 2023

Conversation

Builditluc
Copy link
Owner

Linked Issues #143

Changes made:

  • feat: add Parser, ElementParser and Element traits
  • feat: add Article and Section and implement Article::from_mediawiki
  • feat: add Text element
  • feat: add Header element
  • feat: add MediawikiParagraphParser
  • feat: add MediawikiUnsupportedElementParser
  • feat: add MediawikiParser::new
  • feat: finish basic parsing
  • feat: add Header element
  • feat: implement link element
  • feat: add italic and bold element parser
  • feat: add MediawikiListParser

Changes made:
* feat: add mediawiki api struct with search working
* feat: implement article_from_title for api handler
* refactor: use map() for retrieving the article sections
* feat: implement article_from_id for api handler
* feat: implement search_at_offset for api handler
* refactor: remove Search::origin and make result structs from api handler public
* style: clean up the search offset fetching in search_from_json
* refactor: create helper function for creating a query
@Builditluc Builditluc added state: approved This issue or pull request was approved and can be worked on type: refactor Changes to the style and structure of the codebase or project labels Feb 6, 2023
@Builditluc Builditluc self-assigned this Feb 6, 2023
@Builditluc Builditluc merged commit 0ec8a49 into main Feb 6, 2023
@Builditluc Builditluc deleted the 143-create-parser-system branch February 6, 2023 07:09
@Builditluc Builditluc restored the 143-create-parser-system branch February 6, 2023 07:09
@Builditluc Builditluc removed state: approved This issue or pull request was approved and can be worked on type: refactor Changes to the style and structure of the codebase or project labels Feb 6, 2023
@Builditluc Builditluc added the ignore-release This won't be included in the release notes label Feb 6, 2023
@Builditluc Builditluc deleted the 143-create-parser-system branch February 6, 2023 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ignore-release This won't be included in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant