Skip to content

Fixing some things#5

Open
graysonkoy wants to merge 2 commits into
bacebu4:masterfrom
graysonkoy:fix/classes-wrapper
Open

Fixing some things#5
graysonkoy wants to merge 2 commits into
bacebu4:masterfrom
graysonkoy:fix/classes-wrapper

Conversation

@graysonkoy
Copy link
Copy Markdown

@graysonkoy graysonkoy commented Jan 16, 2026

Currently, reports output like this:

<packages >
	<package ...>
		<class ...>
			<methods >
			</lines>
		</class>
		<class ...>
			<methods >
			</lines>
		</class>
	</package>
</packages>

They should look like this:

<packages>
	<package ...>
		<classes>
			<class ...>
				<methods>
				</lines>
			</class>
			<class ...>
				<methods>
				</lines>
			</class>
		</classes>
	</package>
</packages>

Differences:

  • There should be a <classes> tag surrounding the <class> tags
  • There shouldn't be a trailing space following tag names when there aren't any attributes

This PR fixes these things.

@graysonkoy graysonkoy changed the title fix: add <classes> wrapper Fixing some things Jan 16, 2026
@bacebu4
Copy link
Copy Markdown
Owner

bacebu4 commented Jan 25, 2026

Thanks for the contribution!

Sorry for late reply. I'll review this change within a day and release it if everything will work correctly

bacebu4 pushed a commit that referenced this pull request Mar 16, 2026
Reviewed PR #5 by graysonkoy which adds <classes> wrapper element
and removes trailing spaces in tags with no attributes. Both fixes
align output with Cobertura XML DTD. Tests pass. Approved.

https://claude.ai/code/session_012F7zQj1i5DTz688r7dRd2D
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