Skip to content

Remove deprecated class XDMFormatters#350

Merged
kevinlind merged 2 commits intoadobe:devfrom
kevinlind:remove-formatters
May 31, 2023
Merged

Remove deprecated class XDMFormatters#350
kevinlind merged 2 commits intoadobe:devfrom
kevinlind:remove-formatters

Conversation

@kevinlind
Copy link
Copy Markdown
Contributor

Description

Remove the deprecated public class XDMFormatters for the 2.0.0 release.

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@kevinlind kevinlind requested a review from emdobrin May 31, 2023 19:14
@codecov
Copy link
Copy Markdown

codecov Bot commented May 31, 2023

Codecov Report

Merging #350 (a0a5990) into dev (14ae9c2) will increase coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev     #350      +/-   ##
==========================================
+ Coverage   96.61%   96.68%   +0.07%     
==========================================
  Files          28       27       -1     
  Lines        1650     1626      -24     
==========================================
- Hits         1594     1572      -22     
+ Misses         56       54       -2     

Comment thread Tests/UnitTests/XDMFormattersTests.swift
@kevinlind kevinlind merged commit 7f530ce into adobe:dev May 31, 2023
@kevinlind kevinlind deleted the remove-formatters branch May 31, 2023 22:45
@emdobrin emdobrin added this to the v4.0.0 milestone Jun 2, 2023
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