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

Apply new Shipkit Changelog plugin properties #2163

Merged
merged 2 commits into from
Dec 31, 2020

Conversation

shestee
Copy link
Contributor

@shestee shestee commented Dec 30, 2020

Two new properties has been recently added to Shipkit Changelog plugin that is used by Mockito:

  1. newTagRevision eliminates problem with wrong tagging, when release is tagged with revision reference that the release does not come from, but is the latest one on master branch. To get the commit SHA that triggered the workflow 'GITHUB_SHA' env var is used.
  2. githubToken is used both to fetch and post via GH API, instead of readOnlyToken and writeToken. As the token assigned to 'githubToken' should have write access to the repository, it should be supplied by the env variable to keep things safe. Using this property simplifies configuration. Deprecated properties should be replaced with the new one.

shestee and others added 2 commits December 30, 2020 22:46
Two new properties has been recently added to Shipkit Changelog plugin:
1. 'newTagRevision' eliminates problem with wrong tagging, when release is tagged with revision reference that the release doesn't come from, but is the latest one on master branch. To get the commit SHA that triggered the workflow 'GITHUB_SHA' env var is used.
2. 'githubToken' is used to both to fetch and post, instead of 'readOnlyToken' and 'writeToken'. As the token grants write access to the repository to keep things safe it should be supplied by the env variable. Using this property simplifies configuration. Deprecated properties should be replaced with the new one.
@codecov-io
Copy link

Codecov Report

Merging #2163 (5f05808) into release/3.x (a6b0e7c) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff               @@
##             release/3.x    #2163   +/-   ##
==============================================
  Coverage          84.86%   84.86%           
  Complexity          2720     2720           
==============================================
  Files                325      325           
  Lines               8277     8277           
  Branches             989      989           
==============================================
  Hits                7024     7024           
  Misses               981      981           
  Partials             272      272           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a6b0e7c...6ade10a. Read the comment docs.

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

3 participants