Skip to content

Conversation

agboom
Copy link

@agboom agboom commented Oct 7, 2024

Hi, this is still a draft and my first attempt at migration. I was looking to contribute to the sbt 2.x plugin migration and found sbt-git to be one that's used often.

I had to disable sbt-ci-release for now, because it doesn't support 2.x yet and uses sbt-git itself as well. So I think we need to do some kind of bootstrap without sbt-ci-release before we can migrate that one too?

Let me know what you think.

Edit: apparently sbt-ci-release was migrated to 2.x just now, so I'll update my PR accordingly

Edit: it seems to be unable to resolve sbt-ci-release 1.6.1+13-1458cb0b-SNAPSHOT, I'm probably missing something here

Comment on lines -20 to +22
if(ConsoleLogger.formatEnabled) Seq("GIT_PAGER_IN_USE" -> "1")
if(ConsoleAppender.formatEnabledInEnv) Seq("GIT_PAGER_IN_USE" -> "1")
Copy link
Author

Choose a reason for hiding this comment

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

Both methods are deprecated, the latter points to Terminal.isColorEnabled, but that one's private. I'm not sure if these will be removed in the final 2.x, but for now I couldn't find a suitable replacement.

Comment on lines +4 to +5
> git config core.hookspath /dev/null
> git config commit.gpgsign false
Copy link
Author

@agboom agboom Oct 7, 2024

Choose a reason for hiding this comment

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

Some scripted tests started to take a very long time. I figured it was waiting for a GPG password prompt, so I added these lines I added before in a plugin of my own. Now they quickly pass on my machine.

@mzuehlke
Copy link
Contributor

mzuehlke commented Oct 8, 2024

there is #256

@agboom
Copy link
Author

agboom commented Oct 8, 2024

Let's continue in #256

@agboom agboom closed this Oct 8, 2024
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