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.

Sign up for GitHub

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

Jump to bottom

Remove old and unmaintained junit extension #10668

Merged
merged 1 commit into from
Oct 9, 2023
Merged

Remove old and unmaintained junit extension #10668

merged 1 commit into from
Oct 9, 2023

Conversation

nickvergessen
Copy link
Member

@nickvergessen nickvergessen commented Oct 9, 2023 •
edited
Loading

☑️ Resolves

  • Currently CI always yields:
Deprecated: Creation of dynamic property DOMDocument::$preserveWhitespace is deprecated in /home/runner/work/spreed/spreed/apps/spreed/tests/integration/vendor/jarnaiz/behat-junit-formatter/src/Formatter/JUnitFormatter.php on line 314

And it will break at some point.

🛠️ API Checklist

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not possible
  • 📘 API documentation in docs/ has been updated or is not required
  • 🔖 Capability is added or not needed

@nickvergessen nickvergessen added 3. to review feature: api 🛠️ OCS API for conversations, chats and participants technical debt labels Oct 9, 2023
@nickvergessen nickvergessen added this to the 💙 Next Major (28) milestone Oct 9, 2023
@nickvergessen nickvergessen requested a review from danxuliu October 9, 2023 13:29
@nickvergessen nickvergessen self-assigned this Oct 9, 2023
@nickvergessen
Remove old and unmaintained junit extension
8813244 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen changed the title Remove unused, old and unmaintained junit extension Remove old and unmaintained junit extension Oct 9, 2023
@nickvergessen nickvergessen force-pushed the techdebt/noid/remove-unused-dependency branch from 1fae979 to 8813244 Compare October 9, 2023 13:36
@nickvergessen nickvergessen merged commit a79e42a into master Oct 9, 2023
40 checks passed
@nickvergessen nickvergessen deleted the techdebt/noid/remove-unused-dependency branch October 9, 2023 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danxuliu danxuliu danxuliu approved these changes

Assignees

@nickvergessen nickvergessen

Labels
3. to review feature: api 🛠️ OCS API for conversations, chats and participants technical debt
Projects
None yet
Milestone
v18.0.0-beta.1
Development

Successfully merging this pull request may close these issues.
2 participants
@nickvergessen @danxuliu