Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

#445 - clean up repo by removing maven #621

Merged
merged 1 commit into from
Feb 16, 2020
Merged

#445 - clean up repo by removing maven #621

merged 1 commit into from
Feb 16, 2020

Conversation

jaredsburrows
Copy link
Contributor

@jaredsburrows jaredsburrows commented Oct 17, 2019

@shashachu
Copy link
Contributor

Thanks @jaredsburrows! I verified that after updating to 0.36.0-SNAPSHOT (which has this fix #617), I don't have to make any changes to my gradle files, so it should be safe to start making releases via gradle.

.editorconfig Outdated
@@ -13,9 +13,5 @@ insert_final_newline = true
[*.{java,kt,kts,scala,rs,xml,kt.spec,kts.spec}]
indent_size = 4

[{Makefile,*.go}]
Copy link
Contributor

Choose a reason for hiding this comment

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

This seems unrelated?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Reverted.

.announce Outdated
@@ -1,44 +0,0 @@
#!/bin/bash -e
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think this file is used to update version in README.md and on the ktlint.github.io site.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Reverted

@Tapchicoma
Copy link
Collaborator

Also maven contains step to automatically create a release notes on Github using chandler tool.

@jaredsburrows
Copy link
Contributor Author

@Tapchicoma Should we port the maven step/command over to Gradle?

@Tapchicoma
Copy link
Collaborator

@Tapchicoma Should we port the maven step/command over to Gradle?

Yes, would be nice. Then Maven could be removed after test release with Gradle.

@jaredsburrows
Copy link
Contributor Author

@Tapchicoma Is there a plugin for this?

@Tapchicoma
Copy link
Collaborator

As far as I know chandler itself is not maintained anymore. Propose replacement is https://github.com/apps/release-drafter app. No idea though how could it be integrated with Gradle.

@shashachu
Copy link
Contributor

I'm okay without chandler, but the maven steps also include a deploy to GitHub which I'd like to not lose. I had to manually upload and generate the signature.

@msfjarvis
Copy link

This plugin seems to work just fine, might be worth evaluating for the use case here.

@Tapchicoma
Copy link
Collaborator

@msfjarvis thank you, will take a look 👍

@Tapchicoma
Copy link
Collaborator

@jaredsburrows could you resolve conflict in pom.xml file and update PR with latest changes from master?

Chandler tool could be replaced with plugin proposed by @msfjarvis and I add in a separate PR. So this PR could be merged as is.

@jaredsburrows
Copy link
Contributor Author

@Tapchicoma done!

@Tapchicoma Tapchicoma merged commit fb0b8b4 into pinterest:master Feb 16, 2020
@Goooler Goooler mentioned this pull request Sep 1, 2022
6 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants