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

improvement: add project's java info to doctor #6226

Merged
merged 3 commits into from
Apr 2, 2024

Conversation

kasiaMarek
Copy link
Contributor

Add information about java used for project to doctor.

@ckipp01
Copy link
Member

ckipp01 commented Mar 15, 2024

Could you also make sure it's included in the json output? I believe right now it's only the html.

@ckipp01
Copy link
Member

ckipp01 commented Mar 15, 2024

O, also since the format changed we should bump the DoctorResults.version as well.

@kasiaMarek
Copy link
Contributor Author

Could you also make sure it's included in the json output?

Thanks, I seem to always forget to add the field serialization in the toJson method.

@kasiaMarek kasiaMarek requested a review from tgodzik April 2, 2024 10:19
@kasiaMarek kasiaMarek merged commit 73a272e into scalameta:main Apr 2, 2024
25 of 26 checks passed
@ckipp01 ckipp01 added the affects clients Use this if you are adding a new setting or making a change that will affect clients. label Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects clients Use this if you are adding a new setting or making a change that will affect clients.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants