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

[TASK] Use eliashaeussler/valinor-xml to map XML sitemaps #379

Merged
merged 1 commit into from
Jun 14, 2024

Conversation

eliashaeussler
Copy link
Owner

This PR replaces the business logic of mapping from XML sitemaps to objects by using the new eliashaeussler/valinor-xml library.

@eliashaeussler eliashaeussler added the maintenance Code base is being maintained label Jun 14, 2024
@eliashaeussler eliashaeussler self-assigned this Jun 14, 2024
Copy link

codeclimate bot commented Jun 14, 2024

Code Climate has analyzed commit 8e53d16 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 93.9% (0.0% change).

View more on Code Climate.

@coveralls
Copy link
Collaborator

coveralls commented Jun 14, 2024

Pull Request Test Coverage Report for Build 9518944687

Details

  • 2 of 2 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.1%) to 93.991%

Totals Coverage Status
Change from base Build 9510801951: -0.1%
Covered Lines: 1392
Relevant Lines: 1481

💛 - Coveralls

@eliashaeussler eliashaeussler changed the title [TASK] Use eliashaeussler/valinor-xml to map XML sitemaps [TASK] Use eliashaeussler/valinor-xml to map XML sitemaps Jun 14, 2024
@eliashaeussler eliashaeussler merged commit 85a1ceb into main Jun 14, 2024
16 checks passed
@eliashaeussler eliashaeussler deleted the task/valinor-xml branch June 14, 2024 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Code base is being maintained
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants