Skip to content

FINERACT-821 Added and Enforced Typename checkstyle#920

Merged
vorburger merged 1 commit intoapache:developfrom
thesmallstar:checkstyle-typename
May 22, 2020
Merged

FINERACT-821 Added and Enforced Typename checkstyle#920
vorburger merged 1 commit intoapache:developfrom
thesmallstar:checkstyle-typename

Conversation

@thesmallstar
Copy link
Member

Refer: https://issues.apache.org/jira/browse/FINERACT-821
Needs a rebase, please do not merge this I will review this once again before any of the mentors can take a look :)

@thesmallstar thesmallstar force-pushed the checkstyle-typename branch from 837f549 to f2014f6 Compare May 21, 2020 22:02
@vorburger
Copy link
Member

before any of the mentors can take a look :)

@thesmallstar you can use Mark as Draft (on the right) to "signal" that a PR is Work-In-Progress (WIP)

@thesmallstar thesmallstar marked this pull request as draft May 21, 2020 22:22
@thesmallstar thesmallstar force-pushed the checkstyle-typename branch from f2014f6 to 46298ae Compare May 21, 2020 22:26
@thesmallstar thesmallstar marked this pull request as ready for review May 22, 2020 02:19
@thesmallstar
Copy link
Member Author

@vorburger ready for your review.

Copy link
Contributor

@xurror xurror left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's alot of files to changes. Great work on this one. LGTM

Copy link
Member

@vorburger vorburger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vorburger
Copy link
Member

@thesmallstar resolve merge conflicts please

@thesmallstar thesmallstar force-pushed the checkstyle-typename branch from 46298ae to 3f8fc18 Compare May 22, 2020 10:45
@vorburger vorburger merged commit baa1177 into apache:develop May 22, 2020
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.

3 participants