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 XML syntax error in freemarker manifest #1054

Merged
merged 1 commit into from
Jan 22, 2025

Conversation

jonahgraham
Copy link
Member

The extra " inside XML property was actually being processed properly by freemarker (surprisingly!), but XML editors/viewers reported an error.

The extra " inside XML property was actually being processed properly
by freemarker (surprisingly!), but XML editors/viewers reported an
error.
Copy link

Test Results

   601 files  ±0     601 suites  ±0   13m 12s ⏱️ +19s
10 200 tests ±0  10 177 ✅ ±0  23 💤 ±0  0 ❌ ±0 
10 238 runs  ±0  10 215 ✅ ±0  23 💤 ±0  0 ❌ ±0 

Results for commit 4bc0d1c. ± Comparison against base commit 9e04dc5.

@jonahgraham jonahgraham marked this pull request as draft January 22, 2025 19:09
@jonahgraham
Copy link
Member Author

I want to make sure the new check added in #1053 fails before I merge this fix.

@jonahgraham jonahgraham marked this pull request as ready for review January 22, 2025 21:39
@jonahgraham jonahgraham merged commit 9c4ace1 into eclipse-cdt:main Jan 22, 2025
5 checks passed
@jonahgraham jonahgraham deleted the fix-xml branch January 22, 2025 21:39
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.

1 participant