Skip to content
This repository has been archived by the owner on Apr 22, 2024. It is now read-only.

[PIR-96] Add Google Java formatting to all 3 projects #44

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

[PIR-96] Add Google Java formatting to all 3 projects #44

wants to merge 5 commits into from

Conversation

julianmclain
Copy link

@julianmclain julianmclain commented Oct 1, 2020 •
edited
Loading

Status

WIP

Jira Ticket(s)

Description

  • Adds a descriptive doc comment for all data mapping methods in the IterableExtension (where data processing happens)
  • Adds google-java formatting as a Gradle plugin using AOSP style. Right now, it's set to ignore formatting validation failures on build.

Testing

  • Unit tested (including negative cases)
  • Integration tested
  • Manually tested with the staging test tile

@julianmclain julianmclain self-assigned this Oct 1, 2020
@vbabenkoru vbabenkoru removed their request for review February 3, 2022 23:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@davidchen-iterable davidchen-iterable davidchen-iterable approved these changes

@marlarabin marlarabin Awaiting requested review from marlarabin

Assignees

@julianmclain julianmclain

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@julianmclain @davidchen-iterable