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

fix: handle-empty-property-group #88

Merged
merged 1 commit into from
Jan 20, 2025

Conversation

37IulianPopovici
Copy link
Contributor

Some users might use propertyGroups like:

  <PropertyGroup>
  </PropertyGroup>

This PR adds handling for that case, and returns the usual empty array instead of an error.

@37IulianPopovici 37IulianPopovici self-assigned this Jan 16, 2025
@37IulianPopovici 37IulianPopovici requested a review from a team as a code owner January 16, 2025 12:18
@CLAassistant
Copy link

CLAassistant commented Jan 16, 2025

CLA assistant check
All committers have signed the CLA.

@37IulianPopovici 37IulianPopovici force-pushed the fix/handle-empty-property-group-case branch from 1e22880 to 384c222 Compare January 16, 2025 12:19
@37IulianPopovici 37IulianPopovici force-pushed the fix/handle-empty-property-group-case branch from 384c222 to 5d5e4ff Compare January 20, 2025 08:20
@37IulianPopovici 37IulianPopovici merged commit 5d65662 into master Jan 20, 2025
3 checks passed
@37IulianPopovici 37IulianPopovici deleted the fix/handle-empty-property-group-case branch January 20, 2025 08:28
@snyksec
Copy link

snyksec commented Jan 20, 2025

🎉 This PR is included in version 5.6.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants