Skip to content

Fixed debug script#702

Merged
RyRy79261 merged 12 commits intomainfrom
ryan/527-debug-script
Nov 6, 2023
Merged

Fixed debug script#702
RyRy79261 merged 12 commits intomainfrom
ryan/527-debug-script

Conversation

@RyRy79261
Copy link
Copy Markdown
Contributor

closes #527

@RyRy79261 RyRy79261 marked this pull request as draft November 6, 2023 11:17
@RyRy79261 RyRy79261 marked this pull request as ready for review November 6, 2023 11:17
@auto-assign auto-assign bot requested a review from kantagara November 6, 2023 11:17
robGG1997 and others added 5 commits November 6, 2023 13:16
* 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>
…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
# Conflicts:
#	Packages/io.chainsafe.web3-unity/Runtime/Libraries/ChainSafe.Gaming.Unity.dll
@robGG1997 robGG1997 added 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 13:01
@robGG1997 robGG1997 marked this pull request as ready for review November 6, 2023 13:01
@auto-assign auto-assign bot requested a review from kantagara November 6, 2023 13:01
@RyRy79261 RyRy79261 removed the ready-to-merge Ready to Merge PR - this'll trigger required checks label Nov 6, 2023
@RyRy79261 RyRy79261 marked this pull request as draft November 6, 2023 13:05
@RyRy79261 RyRy79261 marked this pull request as ready for review November 6, 2023 13:06
@RyRy79261 RyRy79261 added the ready-to-merge Ready to Merge PR - this'll trigger required checks label Nov 6, 2023
@RyRy79261 RyRy79261 requested a review from kalambet November 6, 2023 14:02
@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
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!

@RyRy79261 RyRy79261 merged commit 2e2b733 into main Nov 6, 2023
@RyRy79261 RyRy79261 deleted the ryan/527-debug-script branch November 6, 2023 15:18
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
* 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.

Debug publish script no longer functional

6 participants