Skip to content

[HUDI-2263] add steps for checkstyle#3385

Closed
data-storyteller wants to merge 2 commits intoapache:asf-sitefrom
data-storyteller:HUDI-2263-checkstyle-guide
Closed

[HUDI-2263] add steps for checkstyle#3385
data-storyteller wants to merge 2 commits intoapache:asf-sitefrom
data-storyteller:HUDI-2263-checkstyle-guide

Conversation

@data-storyteller
Copy link
Contributor

@data-storyteller data-storyteller commented Aug 3, 2021

Tips

What is the purpose of the pull request

This pull request provides steps to add hudi checkstyle file to IDE.

Brief change log

  • Added guide/steps for checkstyle

Verify this pull request

(Please pick either of the following options)

This pull request is a trivial rework / code cleanup without any test coverage.

(or)

This pull request is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

  • Manually verified the checkstyle steps.

Committer checklist

  • Has a corresponding JIRA in PR title & commit

  • Commit message is descriptive of the change

  • CI is green

  • Necessary doc changes done or have another open PR

  • For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@vinothchandar
Copy link
Member

@satishmittal1111 can we retarget this on the new site docs. We just moved to docusaurus

# Conflicts:
#	content/assets/images/IDE_setup_checkstyle_1.png
#	content/assets/images/IDE_setup_checkstyle_2.png
#	content/assets/images/IDE_setup_checkstyle_3.png
#	content/assets/images/IDE_setup_checkstyle_4.png
#	docs/_pages/contributing.md
@data-storyteller
Copy link
Contributor Author

Created new PR #3462 with docusaurus changes.

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.

2 participants