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

fix: specify node18, semantic release with exec and git plugins #808

Merged
merged 11 commits into from
Aug 1, 2023

Conversation

FrankFlitton
Copy link
Contributor

@FrankFlitton FrankFlitton commented Jul 27, 2023

TL;DR

Unblock semantic release errors

Type

  • Bug Fix
  • Feature
  • Plugin

Are all requirements met?

  • Code completed
  • Smoke tested
  • Unit tests added
  • Code documentation added
  • Any pending items have an associated Issue

Complete description

Bypass failing @semantic-release/npm failure release pipeline failure by recreating npm versions with @semantic-release/exec and @semantic-release/git.

Updated other parts of the config to match latest versions of semantic-release docs.

Added missing deps for easier reproducibility and debugging.

@FrankFlitton FrankFlitton marked this pull request as ready for review July 27, 2023 21:42
@codecov
Copy link

codecov bot commented Jul 27, 2023

Codecov Report

Merging #808 (a0975cd) into master (8b5cb3f) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master     #808   +/-   ##
=======================================
  Coverage   63.37%   63.37%           
=======================================
  Files         527      527           
  Lines       13362    13362           
  Branches     2540     2540           
=======================================
  Hits         8468     8468           
  Misses       4894     4894           
Files Changed Coverage Δ
...ExecutionDetails/ExecutionDetailsAppBarContent.tsx 86.74% <ø> (ø)
...ions/TerminateExecution/TerminateExecutionForm.tsx 54.16% <0.00%> (ø)
...nsole/src/components/LaunchPlan/LaunchPlanLink.tsx 100.00% <ø> (ø)
...components/Literals/Scalar/ProtobufStructValue.tsx 69.69% <0.00%> (ø)
...nsole/src/components/Navigation/TopLevelLayout.tsx 0.00% <ø> (ø)
...es/console/src/components/common/ErrorBoundary.tsx 65.62% <ø> (ø)
.../src/components/common/ExpandableMonospaceText.tsx 48.00% <ø> (ø)
.../src/components/common/ScrollableMonospaceText.tsx 57.89% <ø> (ø)

Signed-off-by: Frank Flitton <[email protected]>
Signed-off-by: Frank Flitton <[email protected]>
Signed-off-by: Frank Flitton <[email protected]>
Signed-off-by: Frank Flitton <[email protected]>
Signed-off-by: Frank Flitton <[email protected]>
Signed-off-by: Frank Flitton <[email protected]>
@FrankFlitton FrankFlitton changed the title chore: specify node18, add dev deps fix: specify node18, semantic release with exec and git plugins Jul 28, 2023
CHANGELOG.md Outdated Show resolved Hide resolved
Dockerfile Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
Signed-off-by: Frank Flitton <[email protected]>
Signed-off-by: Frank Flitton <[email protected]>
Signed-off-by: Frank Flitton <[email protected]>
const packageJson = JSON.parse(fs.readFileSync('./package.json', 'utf8'));
packageJson.version = version;

console.log('Writing new package.json contents.');
Copy link
Contributor

Choose a reason for hiding this comment

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

is this dupe work of make update_npmversion (makefile ln 58)

Copy link
Contributor Author

@FrankFlitton FrankFlitton Jul 31, 2023

Choose a reason for hiding this comment

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

Not exactly.. this script doesn't update the root pacjage.json version which @semantic-release/npm would. I moved the functionality into a bash script inside the /scripts folder. The new script updates both root and /website package.json files when ran.

@FrankFlitton FrankFlitton merged commit 58d5d1a into master Aug 1, 2023
@FrankFlitton FrankFlitton deleted the frank/fix-release-pipeline branch August 1, 2023 20:03
@flyte-bot
Copy link
Collaborator

🎉 This PR is included in version 1.9.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants