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.

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

KtLint: Make functions needed to calculate line / column public #725

Merged
merged 1 commit into from
May 26, 2020

Conversation

sschuberth
Copy link
Contributor

In order to avoid duplicating code, make these functions public for use
by tools that use KtLint programmatically and need to convert an offset
to line / column, like detekt.

@shashachu
Copy link
Contributor

@sschuberth Apologies for the delay. Can you try rebasing and re-pushing your PR? looks like something went wrong with the checks.

In order to avoid duplicating code, make these functions public for use
by tools that use KtLint programmatically and need to convert an offset
to line / column, like detekt.
@sschuberth sschuberth force-pushed the public-offset-to-line-column branch from 3096f92 to a387c0f Compare May 20, 2020 06:00
@sschuberth
Copy link
Contributor Author

Done @shashachu.

@shashachu shashachu merged commit bbb7ac6 into pinterest:master May 26, 2020
@sschuberth sschuberth deleted the public-offset-to-line-column branch May 26, 2020 17:51
sschuberth added a commit to sschuberth/detekt that referenced this pull request Jun 3, 2020
These functions formerly were private but are public now, see
pinterest/ktlint#725.
arturbosch pushed a commit to detekt/detekt that referenced this pull request Jun 3, 2020
* Upgrade ktlint to version 0.37.0

See https://github.com/pinterest/ktlint/releases/tag/0.37.0. Note that
IndentationRule was promoted to standard.

* Use helper functions directly from ktlint instead of copies

These functions formerly were private but are public now, see
pinterest/ktlint#725.
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.

None yet

2 participants