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

#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
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.

4 participants