Skip to content
This repository has been archived by the owner on Jun 5, 2024. It is now read-only.

ci: Updated prepare-release to use conventional commit based releases. #232

Merged
merged 2 commits into from
Dec 21, 2023

Conversation

bizob2828
Copy link
Member

Since we are going to be doing more work in here I updated the release style to our conventional commit based workflow. I also added the validate-pr workflow and updated the PR template to no longer rely on convention in the PR description to derive changelogs.

@bizob2828 bizob2828 added the dev:automation Indicates CI automation label Dec 21, 2023

jobs:
release-notes:
uses: newrelic/node-newrelic/.github/workflows/prep-release.yml@main
with:
release_type: ${{ github.event.inputs.release_type }}
use_new_relase: ${{ vars.USE_NEW_RELEASE }}
Copy link
Member Author

Choose a reason for hiding this comment

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

i set this in the actions variables to true

@mrickard mrickard self-assigned this Dec 21, 2023
Copy link

codecov bot commented Dec 21, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (c1e4c4c) 95.31% compared to head (cc52d37) 95.31%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #232   +/-   ##
=======================================
  Coverage   95.31%   95.31%           
=======================================
  Files          13       13           
  Lines         790      790           
=======================================
  Hits          753      753           
  Misses         37       37           
Flag Coverage Δ
unit-tests-16.x 4.68% <ø> (ø)
unit-tests-18.x 4.68% <ø> (ø)
unit-tests-20.x 4.68% <ø> (ø)
versioned-tests-16.x 95.31% <ø> (ø)
versioned-tests-18.x 95.31% <ø> (ø)
versioned-tests-20.x 95.31% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

mrickard
mrickard previously approved these changes Dec 21, 2023
@CLAassistant
Copy link

CLAassistant commented Dec 21, 2023

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

@jsumners-nr jsumners-nr left a comment

Choose a reason for hiding this comment

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

Looks good to me.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dev:automation Indicates CI automation
Projects
Node.js Engineering Board
  
Done: Issues recently completed
Development

Successfully merging this pull request may close these issues.

None yet

4 participants