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

Upgrade Pitest Git plugins 1.0.1 -> 1.0.2 #380

Merged
merged 5 commits into from
Dec 5, 2022

Conversation

Picnic-Bot
Copy link
Contributor

This PR contains the following updates:

Package Type Update Change
com.groupcdg:pitest-git-plugin build patch 1.0.1 -> 1.0.2
com.groupcdg:pitest-github-maven-plugin build patch 1.0.1 -> 1.0.2
com.groupcdg:pitest-git-maven-plugin build patch 1.0.1 -> 1.0.2

  • If you want to rebase/retry this PR, check this box

@Picnic-Bot
Copy link
Contributor Author

Picnic-Bot commented Dec 2, 2022

Suggested commit message:

Upgrade Pitest Git plugins 1.0.1 -> 1.0.2 (#380)

@Stephan202 Stephan202 added this to the 0.6.0 milestone Dec 2, 2022
Copy link
Member

@Stephan202 Stephan202 left a comment

Choose a reason for hiding this comment

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

Updated the suggested commit message. @rickie shall we tell Renovate to map version.pitest-git to Arcmutate?

Logs show that Arcmutate is still active:

2022-12-02T02:02:03.6181276Z >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>> 
2022-12-02T02:02:03.6182177Z Licenced for use in the development of open source software only.
2022-12-02T02:02:03.6182917Z Expires: 2023-11-07
2022-12-02T02:02:03.6184690Z For use with packages: tech.picnic.errorprone.*
2022-12-02T02:02:03.6185467Z >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>> 

... but I'll quickly push a tweak to check it still reports stuff.

@Stephan202
Copy link
Member

The master-based build that uploads the artifact failed:

2022-12-02T07:15:27.9564416Z ##[group]Run dawidd6/[email protected]
2022-12-02T07:15:27.9564708Z with:
2022-12-02T07:15:27.9564915Z   workflow: 40328986
2022-12-02T07:15:27.9565177Z   name: pitest-reports
2022-12-02T07:15:27.9565431Z   path: ./target/pit-reports-ci
2022-12-02T07:15:27.9565870Z   github_token: ***
2022-12-02T07:15:27.9566105Z   workflow_conclusion: success
2022-12-02T07:15:27.9566391Z   repo: PicnicSupermarket/error-prone-support
2022-12-02T07:15:27.9566662Z   check_artifacts: false
2022-12-02T07:15:27.9566878Z   search_artifacts: false
2022-12-02T07:15:27.9567104Z   skip_unpack: false
2022-12-02T07:15:27.9567331Z   if_no_artifact_found: fail
2022-12-02T07:15:27.9567557Z env:
2022-12-02T07:15:27.9567838Z   JAVA_HOME: /opt/hostedtoolcache/Java_Temurin-Hotspot_jdk/17.0.4-101/x64
2022-12-02T07:15:27.9568214Z   JAVA_HOME_17_X64: /opt/hostedtoolcache/Java_Temurin-Hotspot_jdk/17.0.4-101/x64
2022-12-02T07:15:27.9568488Z ##[endgroup]
2022-12-02T07:15:28.0865941Z ==> Repository: PicnicSupermarket/error-prone-support
2022-12-02T07:15:28.0866999Z ==> Artifact name: pitest-reports
2022-12-02T07:15:28.0867943Z ==> Local path: ./target/pit-reports-ci
2022-12-02T07:15:28.0868362Z ==> Workflow name: 40328986
2022-12-02T07:15:28.0868816Z ==> Workflow conclusion: success
2022-12-02T07:15:39.1604426Z ##[error]Server Error

Doesn't seem Pitest-related, so will restart with debug logging 👀

@Stephan202
Copy link
Member

Nope, failed again. I don't see how this is related to the upgrade, though. Let me downgrade and see what that does 👀

@Stephan202
Copy link
Member

Also failed. Could be a GH-side issue. Let's check back later.

@github-actions
Copy link

github-actions bot commented Dec 2, 2022

  • Surviving mutants in this change: 1
  • Killed mutants in this change: 1
class surviving killed
🧟tech.picnic.errorprone.bugpatterns.AutowiredConstructor 1 1

Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed.

@Picnic-Bot
Copy link
Contributor Author

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.

Warning: custom changes will be lost.

Copy link
Member

@Stephan202 Stephan202 left a comment

Choose a reason for hiding this comment

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

Works again! Will revert the first commit as well.

@github-actions
Copy link

github-actions bot commented Dec 2, 2022

Looks good. No mutations were possible for these changes.
Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed.

@Stephan202
Copy link
Member

@rickie shall we tell Renovate to map version.pitest-git to Arcmutate?

In light of #383, perhaps Pitest Git plugins is a better name; updated the suggested commit message.

@rickie rickie force-pushed the renovate/version.pitest-git branch from bed0b0c to c958849 Compare December 5, 2022 07:45
Copy link
Member

@rickie rickie left a comment

Choose a reason for hiding this comment

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

It's a bit weird to not have any release notes though 🤔.

Suggested commit message LGTM.

Will merge once 🍏.

@github-actions
Copy link

github-actions bot commented Dec 5, 2022

Looks good. No mutations were possible for these changes.
Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed.

@rickie rickie changed the title Upgrade version.pitest-git 1.0.1 -> 1.0.2 Upgrade Pitest Git plugins 1.0.1 -> 1.0.2 Dec 5, 2022
@rickie rickie merged commit 789f8c8 into master Dec 5, 2022
@rickie rickie deleted the renovate/version.pitest-git branch December 5, 2022 08:01
@rickie rickie mentioned this pull request Dec 19, 2022
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants