Skip to content

CI updates [Merge Third] - auto commits on PR then trigger checks manually by labeling PR ready-to-merge and/or ready_for_review (PR out of draft)#710

Merged
robGG1997 merged 26 commits intomainfrom
rob/trigger-tests-with-label-709
Nov 6, 2023
Merged

CI updates [Merge Third] - auto commits on PR then trigger checks manually by labeling PR ready-to-merge and/or ready_for_review (PR out of draft)#710
robGG1997 merged 26 commits intomainfrom
rob/trigger-tests-with-label-709

Conversation

@robGG1997
Copy link
Copy Markdown
Contributor

closes #709

@robGG1997 robGG1997 self-assigned this Nov 6, 2023
@robGG1997 robGG1997 added the ready-to-merge Ready to Merge PR - this'll trigger required checks label Nov 6, 2023
@robGG1997 robGG1997 changed the title CI updates - auto commits on PR then trigger checks manually by labeling PR ready-to-merge CI updates [Merge Third] - auto commits on PR then trigger checks manually by labeling PR ready-to-merge Nov 6, 2023
@robGG1997 robGG1997 force-pushed the rob/trigger-tests-with-label-709 branch from c1d3d4f to cadd08e Compare November 6, 2023 08:52
@robGG1997 robGG1997 removed the ready-to-merge Ready to Merge PR - this'll trigger required checks label Nov 6, 2023
@robGG1997 robGG1997 marked this pull request as draft November 6, 2023 08:54
@robGG1997 robGG1997 marked this pull request as ready for review November 6, 2023 08:54
@robGG1997 robGG1997 added the ready-to-merge Ready to Merge PR - this'll trigger required checks label Nov 6, 2023
@robGG1997 robGG1997 changed the title CI updates [Merge Third] - auto commits on PR then trigger checks manually by labeling PR ready-to-merge CI updates [Merge Third] - auto commits on PR then trigger checks manually by labeling PR ready-to-merge and/or ready_for_review (PR out of draft) Nov 6, 2023
Copy link
Copy Markdown
Member

@kalambet kalambet left a comment

Choose a reason for hiding this comment

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

LGTM!

…w workflow for making auto commits

This is done to avoid a head ref error when trying to make two pushes simultaneously
@robGG1997 robGG1997 force-pushed the rob/trigger-tests-with-label-709 branch from db64e54 to 7655e16 Compare November 6, 2023 11:05
@robGG1997 robGG1997 added dependencies Pull requests that update a dependency file ready-to-merge Ready to Merge PR - this'll trigger required checks and removed ready-to-merge Ready to Merge PR - this'll trigger required checks dependencies Pull requests that update a dependency file labels Nov 6, 2023
@robGG1997 robGG1997 added ready-to-merge Ready to Merge PR - this'll trigger required checks and removed ready-to-merge Ready to Merge PR - this'll trigger required checks labels Nov 6, 2023
Copy link
Copy Markdown
Contributor

@kantagara kantagara left a comment

Choose a reason for hiding this comment

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

LGTM

@robGG1997 robGG1997 merged commit cb05b97 into main Nov 6, 2023
@robGG1997 robGG1997 deleted the rob/trigger-tests-with-label-709 branch November 6, 2023 11:57
RyRy79261 pushed a commit that referenced this pull request Nov 6, 2023
…ually by labeling PR `ready-to-merge` and/or ready_for_review (PR out of draft) (#710)

* Revert "automated duplicating imported samples modifications (#595)"

This reverts commit a555894.

* auto commit on PR then run tests manually on labeling with ready-to-merge

* trigger checks when PR is out of draft and ready for review too

* reverted auto commit actions triggers to workflow call and added a new workflow for making auto commits
This is done to avoid a head ref error when trying to make two pushes simultaneously

* new workflow for auto commiting

* small typo fix
RyRy79261 added a commit that referenced this pull request Nov 6, 2023
* Fixed debug script

* Corrected directory

* Re-ran scripts

* Lootboxes Sample Fix [Merge Second] (#704)

* Revert "automated duplicating imported samples modifications (#595)"

This reverts commit a555894.

* small revert to login scene load changes and lootboxes sample duplicate added to CI

* Auto-duplicate Packages Samples

* Goerli check modified/optimized to be easily replaced with sepolia or other chain

* Auto-duplicate Packages Samples

* Logo.psd file in sprites removed

* Auto-duplicate Packages Samples

* Published Solution Dependencies to Package Libraries as DLLs

---------

Co-authored-by: robGG1997 <robGG1997@users.noreply.github.com>

* CI updates [Merge Third] - auto commits on PR then trigger checks manually by labeling PR `ready-to-merge` and/or ready_for_review (PR out of draft) (#710)

* Revert "automated duplicating imported samples modifications (#595)"

This reverts commit a555894.

* auto commit on PR then run tests manually on labeling with ready-to-merge

* trigger checks when PR is out of draft and ready for review too

* reverted auto commit actions triggers to workflow call and added a new workflow for making auto commits
This is done to avoid a head ref error when trying to make two pushes simultaneously

* new workflow for auto commiting

* small typo fix

* Merged main & rebuilt

* Published Solution Dependencies to Package Libraries as DLLs

* Update scripts/debug-publish-to-unity-package.bat

* Fixed

---------

Co-authored-by: robGG1997 <142783680+robGG1997@users.noreply.github.com>
Co-authored-by: robGG1997 <robGG1997@users.noreply.github.com>
Co-authored-by: RyRy79261 <RyRy79261@users.noreply.github.com>
rob1997 pushed a commit that referenced this pull request Jan 16, 2025
…ually by labeling PR `ready-to-merge` and/or ready_for_review (PR out of draft) (#710)

* Revert "automated duplicating imported samples modifications (#595)"

This reverts commit a555894.

* auto commit on PR then run tests manually on labeling with ready-to-merge

* trigger checks when PR is out of draft and ready for review too

* reverted auto commit actions triggers to workflow call and added a new workflow for making auto commits
This is done to avoid a head ref error when trying to make two pushes simultaneously

* new workflow for auto commiting

* small typo fix
rob1997 pushed a commit that referenced this pull request Jan 16, 2025
* Fixed debug script

* Corrected directory

* Re-ran scripts

* Lootboxes Sample Fix [Merge Second] (#704)

* Revert "automated duplicating imported samples modifications (#595)"

This reverts commit a555894.

* small revert to login scene load changes and lootboxes sample duplicate added to CI

* Auto-duplicate Packages Samples

* Goerli check modified/optimized to be easily replaced with sepolia or other chain

* Auto-duplicate Packages Samples

* Logo.psd file in sprites removed

* Auto-duplicate Packages Samples

* Published Solution Dependencies to Package Libraries as DLLs

---------

Co-authored-by: robGG1997 <robGG1997@users.noreply.github.com>

* CI updates [Merge Third] - auto commits on PR then trigger checks manually by labeling PR `ready-to-merge` and/or ready_for_review (PR out of draft) (#710)

* Revert "automated duplicating imported samples modifications (#595)"

This reverts commit a555894.

* auto commit on PR then run tests manually on labeling with ready-to-merge

* trigger checks when PR is out of draft and ready for review too

* reverted auto commit actions triggers to workflow call and added a new workflow for making auto commits
This is done to avoid a head ref error when trying to make two pushes simultaneously

* new workflow for auto commiting

* small typo fix

* Merged main & rebuilt

* Published Solution Dependencies to Package Libraries as DLLs

* Update scripts/debug-publish-to-unity-package.bat

* Fixed

---------

Co-authored-by: robGG1997 <142783680+robGG1997@users.noreply.github.com>
Co-authored-by: robGG1997 <robGG1997@users.noreply.github.com>
Co-authored-by: RyRy79261 <RyRy79261@users.noreply.github.com>
sergeypanin1994 pushed a commit to sergeypanin1994/web3.unity that referenced this pull request Mar 16, 2025
…ually by labeling PR `ready-to-merge` and/or ready_for_review (PR out of draft) (ChainSafe#710)

* Revert "automated duplicating imported samples modifications (ChainSafe#595)"

This reverts commit a555894.

* auto commit on PR then run tests manually on labeling with ready-to-merge

* trigger checks when PR is out of draft and ready for review too

* reverted auto commit actions triggers to workflow call and added a new workflow for making auto commits
This is done to avoid a head ref error when trying to make two pushes simultaneously

* new workflow for auto commiting

* small typo fix
sergeypanin1994 pushed a commit to sergeypanin1994/web3.unity that referenced this pull request Mar 16, 2025
* Fixed debug script

* Corrected directory

* Re-ran scripts

* Lootboxes Sample Fix [Merge Second] (ChainSafe#704)

* Revert "automated duplicating imported samples modifications (ChainSafe#595)"

This reverts commit a555894.

* small revert to login scene load changes and lootboxes sample duplicate added to CI

* Auto-duplicate Packages Samples

* Goerli check modified/optimized to be easily replaced with sepolia or other chain

* Auto-duplicate Packages Samples

* Logo.psd file in sprites removed

* Auto-duplicate Packages Samples

* Published Solution Dependencies to Package Libraries as DLLs

---------

Co-authored-by: robGG1997 <robGG1997@users.noreply.github.com>

* CI updates [Merge Third] - auto commits on PR then trigger checks manually by labeling PR `ready-to-merge` and/or ready_for_review (PR out of draft) (ChainSafe#710)

* Revert "automated duplicating imported samples modifications (ChainSafe#595)"

This reverts commit a555894.

* auto commit on PR then run tests manually on labeling with ready-to-merge

* trigger checks when PR is out of draft and ready for review too

* reverted auto commit actions triggers to workflow call and added a new workflow for making auto commits
This is done to avoid a head ref error when trying to make two pushes simultaneously

* new workflow for auto commiting

* small typo fix

* Merged main & rebuilt

* Published Solution Dependencies to Package Libraries as DLLs

* Update scripts/debug-publish-to-unity-package.bat

* Fixed

---------

Co-authored-by: robGG1997 <142783680+robGG1997@users.noreply.github.com>
Co-authored-by: robGG1997 <robGG1997@users.noreply.github.com>
Co-authored-by: RyRy79261 <RyRy79261@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Ready to Merge PR - this'll trigger required checks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use labels and/or ready_for_review (PR out of draft) to trigger merge checks/tests

4 participants