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

[API view] Formatting issue in Java API view #4052

Closed
srnagar opened this issue Sep 1, 2022 · 4 comments · Fixed by #4287
Closed

[API view] Formatting issue in Java API view #4052

srnagar opened this issue Sep 1, 2022 · 4 comments · Fixed by #4287
Assignees
@Vailorium
Labels
APIView massey

Comments

@srnagar
Copy link
Member

srnagar commented Sep 1, 2022

https://apiviewstaging.azurewebsites.net/Assemblies/Review/9f0252f1f1264c61be1fe517a0d8d667

  • The class annotation has additional space.
  • The class ending parenthesis indentation does not align with class declaration indentation
    image
@JonathanGiles JonathanGiles moved this to 🆕 New in ApiView Sep 1, 2022
@JonathanGiles
Copy link
Member

JonathanGiles commented Sep 5, 2022 •
edited
Loading

Looks to be a regression introduced by #3941, @Vailorium is going to look into it.

@JonathanGiles JonathanGiles removed their assignment Sep 5, 2022
@JonathanGiles JonathanGiles moved this from 🆕 New to 📋 Backlog in ApiView Sep 5, 2022
@Vailorium
Copy link
Contributor

@JonathanGiles

@JonathanGiles JonathanGiles moved this from 📋 Backlog to 🏗 In progress in ApiView Sep 20, 2022
@JonathanGiles JonathanGiles mentioned this issue Sep 22, 2022
Merged
@Vailorium
Copy link
Contributor

This appears to be working fine on the main branch - it was probably resolved with #4091

Repository owner moved this from 🏗 In progress to ✅ Done in ApiView Oct 3, 2022
@JonathanGiles JonathanGiles reopened this Oct 4, 2022
@Vailorium
Copy link
Contributor

Vailorium commented Oct 4, 2022 •
edited
Loading

Issue is in the Java parser code, which I hadn't rebuilt since some changes were made. Reopened and looking into this again.

@Vailorium Vailorium mentioned this issue Oct 4, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Vailorium Vailorium

Labels
APIView massey
Projects
ApiView
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Solved formatting issue in Java API view Vailorium/azure-sdk-tools
3 participants
@JonathanGiles @Vailorium @srnagar