GSOC 2025 - Improve Google-Style - July

5 views
Skip to first unread message

Mohit Sharma

unread,
Jul 7, 2025, 9:15:56 AMJul 7
to checkstyle-devel
Monday, July 7 report

What I did:
We successfully merged the following last week:

 Changes:
 1. Google-formatter CI now runs only on both compilable and non-compilable   inputs.

 2. Google IT 'resources-noncompilable' inputs are being compiled against the latest JDKs.

Currently Working on:

Mohit Sharma

unread,
Jul 11, 2025, 6:32:07 AMJul 11
to checkstyle-devel
Thursday, 10 July report
1. Implemented the format-checking function of TextBlockGoogleStyleFormatting Check. Once we make sure there are no false-positives in my implementation so far, I will start working on the indentation part of the check.
The most time-consuming part so far has been resolving build issues, especially with the XdocsPage test.


Mohit Sharma

unread,
Jul 14, 2025, 8:44:32 AMJul 14
to checkstyle-devel
Monday, 14 July report

What I did:
1. Fixed build errors on TextBlockGoogleStyleFormatting Check.
2. Followed up on Javadoc invalid position false-positive for compact constructor issue: https://github.com/checkstyle/checkstyle/pull/17249

Plan for the week:
1. Achieve green CI on TextBlockGoogleStyleFormatting and regression testing.
2. Backlog from prev week: https://github.com/checkstyle/checkstyle/pull/17308. because I've never dealt with Xpath suppressions, I'm finding writing xpath query a little harder and unintuitive.

Mohit Sharma

unread,
Jul 17, 2025, 8:30:51 AMJul 17
to checkstyle-devel
Thursday, 17 July report

What I did:
1. Started Working on indentation function of TextBlockGoogleStyleFormatting: https://github.com/checkstyle/checkstyle/pull/17395
    Blocker: I found that XpathRegression..Tests only support one error per test, but this Check logs two violations at the same time. so I can't get the Xpath tests passed.

2. Add support for ELLIPSIS token in NoWhitespaceBefore module and  suppress false-positives using suppression: https://github.com/checkstyle/checkstyle/pull/17308
3. Add support for TEXT_BLOCK_CONTENT token in IllegalTokenText module: https://github.com/checkstyle/checkstyle/pull/17414

Next Steps:
1. I'm waiting for mentors to review the above work.
2. Resume: https://github.com/checkstyle/checkstyle/pull/17228

Mohit Sharma

unread,
Jul 21, 2025, 9:57:40 AMJul 21
to checkstyle-devel
Monday, 21 July report

Merged PR:
Next Steps:
1. More priority is to reduce blue ticks from coverage section: so I'm currently working on the following whitespace related issues:
2. Add functionality to the XpathRegression..Tests to support multiple violations.

Mohit Sharma

unread,
Jul 24, 2025, 9:30:05 AMJul 24
to checkstyle-devel
Thursday, 24 July report

What I did:
2. Created issue to separately address false-negatives: whitespace between type annotation and array declarator or ellipsis.

Next steps:
1. Work on enforcing whitespace before left curly of empty record blocks, static blocks, interface blocks, annotation blocks, class blocks and enum blocks.

Mohit Sharma

unread,
Jul 28, 2025, 10:39:25 PMJul 28
to checkstyle-devel
Monday, 28 July report

This week's major focus is going to be on updating google style coverage tables on the website. Some of the changes have already been merged.


Once the table is updated to the latest version, we will start tackling blue-tick Checks and false-positives.

Mohit Sharma

unread,
Jul 31, 2025, 12:36:54 PMJul 31
to checkstyle-devel
Reply all
Reply to author
Forward
0 new messages