Skip to content

Change master to main#551

Merged
mferrera merged 1 commit intomainfrom
update-master-to-main
Jun 19, 2023
Merged

Change master to main#551
mferrera merged 1 commit intomainfrom
update-master-to-main

Conversation

@mferrera
Copy link
Copy Markdown
Collaborator

No description provided.

@mferrera mferrera force-pushed the update-master-to-main branch 2 times, most recently from 8a179d8 to 77b0485 Compare June 16, 2023 18:42
@mferrera mferrera requested a review from alifbe June 19, 2023 05:26
Copy link
Copy Markdown
Collaborator

@alifbe alifbe left a comment

Choose a reason for hiding this comment

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

LGTM 👍

I did a search for master in subscript repo and found this:

args: ["--branch", "master"]
Not sure if this also need to be replaced.

@mferrera mferrera force-pushed the update-master-to-main branch from 77b0485 to 64ccd9a Compare June 19, 2023 05:34
@mferrera
Copy link
Copy Markdown
Collaborator Author

mferrera commented Jun 19, 2023

Nice catch. I fixed it, although the branch protection rules should prevent commits to the default branch from getting upstream anyway so that hook is probably not needed.

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #551 (2119cf4) into main (e5e3f72) will not change coverage.
The diff coverage is n/a.

❗ Current head 2119cf4 differs from pull request most recent head 64ccd9a. Consider uploading reports for the commit 64ccd9a to get more accurate results

@@           Coverage Diff           @@
##             main     #551   +/-   ##
=======================================
  Coverage   86.20%   86.20%           
=======================================
  Files          49       49           
  Lines        7037     7037           
=======================================
  Hits         6066     6066           
  Misses        971      971           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@mferrera mferrera merged commit 0eba2c8 into main Jun 19, 2023
@mferrera mferrera deleted the update-master-to-main branch June 19, 2023 05:51
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.

3 participants