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

add a script to automate bumping the version number #1132

Merged
merged 1 commit into from
Nov 6, 2023
Merged

Conversation

dyc3
Copy link
Owner

@dyc3 dyc3 commented Nov 6, 2023

closes #1122

Copy link

cypress bot commented Nov 6, 2023

1 failed test on run #679 ↗︎

1 48 1 0 Flakiness 0

Details:

Merge e65dc53 into 5c61552...
Project: OpenTogetherTube Commit: 2c27b88a06 ℹ️
Status: Failed Duration: 02:42 💡
Started: Nov 6, 2023 2:12 PM Ended: Nov 6, 2023 2:14 PM
Failed  client/tests/e2e/component/ShareInvite.cy.ts • 1 failed test • Component - electron

View Output Video

Test Artifacts
An uncaught error was detected outside of a test Output Screenshots Video

Review all test suite changes for PR #1132 ↗︎

Copy link

codecov bot commented Nov 6, 2023

Codecov Report

Merging #1132 (e65dc53) into master (5c61552) will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master      #1132   +/-   ##
===========================================
  Coverage   60.4202%   60.4202%           
===========================================
  Files           115        115           
  Lines          9280       9280           
  Branches       1131       1131           
===========================================
  Hits           5607       5607           
  Misses         3673       3673           

@@ -0,0 +1,22 @@
#!/bin/bash

set -exo pipefail
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this line deals with pipeline failure but what does this line do?

Copy link
Owner Author

@dyc3 dyc3 Nov 6, 2023

Choose a reason for hiding this comment

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

It's a one liner that does a couple things for better default behavior. See: https://gist.github.com/mohanpedala/1e2ff5661761d3abd0385e8223e16425

@dyc3 dyc3 merged commit 40909e8 into master Nov 6, 2023
12 of 15 checks passed
@dyc3 dyc3 deleted the set-version-script branch November 6, 2023 15:25
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.

script to set version fields in all package.json and cargo.toml files
2 participants