Skip to content

Conversation

@IddanCF
Copy link

@IddanCF IddanCF commented Nov 28, 2025

Add missing option to set excludeExtensionProp in UrlExportSettings.
Solves #335.

@coveralls
Copy link

coveralls commented Nov 28, 2025

Pull Request Test Coverage Report for Build 19824605659

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 77.921%

Totals Coverage Status
Change from base Build 18306258487: 0.0%
Covered Lines: 655
Relevant Lines: 822

💛 - Coveralls

@0m4r
Copy link
Collaborator

0m4r commented Nov 28, 2025

@IddanCF can you please build the code and add the build files too?

@IddanCF
Copy link
Author

IddanCF commented Dec 1, 2025

@0m4r done! Please check again

Copy link
Collaborator

@0m4r 0m4r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it looks good to me.
@lukasoppermann to provide final approval ;)

@0m4r
Copy link
Collaborator

0m4r commented Dec 12, 2025

@IddanCF a previous PR has been merged.
Would you please take care of the merge conflicts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants