Fix release workflow environment variable definition #4825
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
kamilchodola
pushed a commit
that referenced
this pull request
Oct 27, 2022
LukaszRozmej
added a commit
that referenced
this pull request
Dec 6, 2022
* Implement TraceStore plugin * Add compression to db * Allow unsafe to avoid allocation * Fix EndBlockTrace to be after Applying rewards * Add ByteArrayPoolList * Make ResettableList implement IReadOnlyCollection * Make DbOnTheRocks default to nameless configuration if named not found * Make BlockTracerBase based on ResettableList * Fixes and improvements * Add plugin to pipelines * fix test * Add temporary traces * Revert "Add temporary traces" This reverts commit 9190ed0. * Add exception data to json rpc errors * Increase MaxDepth to 128 * Refactor product versioning and revise Docker files (#4781) * Upgrade version to 14.4 (#4796) Co-authored-by: Kamil Chodoła <kamil@nethermind.io> * Chiado TTD (#4801) * Updating Fast Sync config files (#4803) Co-authored-by: LukaszRozmej <LukaszRozmej@users.noreply.github.com> * Feature: added extra data length checks (#4805) * Added 32 bytes check on extra data Added checks and tests for the ExtraData not to be longer than 32 bytes * Fix workflow to pass parameters to scripts (#4799) * Autorecover when disconnected chain is found (#4816) * Autorecover when disconnected chain is found * Fix tests * Updating Fast Sync config files (#4819) Co-authored-by: MarekM25 <MarekM25@users.noreply.github.com> * Fix environment variable definition (#4825) * Revert "Fix sepolia invalid block (#4694)" (#4838) This reverts commit 8f28e4a. * Bump build version to 14.5 (#4839) Co-authored-by: Kamil Chodoła <kamil@nethermind.io> * fix transition creation * Don't log error, when initialization is stopped (like when syncing previously downloaded blocks from db) * Add VerifySerialized option * Add serialization tests * Move VerifySerialized to TraceSerializer, add MaxDepth config * fix whitespace * fix null handling in TxBroadcaster * Refactor ITraceSerializer * fix test * undo fix * whitespace fix * test fix * fixes * Set chiado and xdai `Merge.SecondsPerSlot=5` (#4880) * Set chiado `Merge.SecondsPerSlot=5` * Update xdai configs with `Merge.SecondsPerSlot=5` This commit is not relevant before the merge. * Updating Fast Sync config files (#4897) Co-authored-by: LukaszRozmej <LukaszRozmej@users.noreply.github.com> * Set C# version to 10.0 (#4881) * Add gnosischain TTD value (#4901) * Add gnosischain TTD value Scheduled for `Dec 5 2022` assuming 5s block times: `8626000000000000000000058750000000000000000000` * Add EnginePort xdai configs * Add xdai TTD test to ChainSpecLoaderTests * Fix runner tests * Override MergePlugin Name and Description fields in AuRaMergePlugin * fix eth_call mix_hash and beneficiary (#4873) * fix eth_call mix_hash and beneficiary * always set MixHash # Conflicts: # src/Nethermind/Nethermind.Facade/BlockchainBridge.cs * Changes the docker password to a token (#4916) * fix: changes the docker password to token * fix: update docker pass for main release workflow * parallelized trace_filter execution per block * Remove unneded filters * Reduce traces memory allocation * fix trace filtering * fix whitespace * Updating Fast Sync config files (#4920) Co-authored-by: LukaszRozmej <LukaszRozmej@users.noreply.github.com> * Bump build version to 14.6 (#4923) Co-authored-by: Kamil Chodoła <kamil@nethermind.io> * Changes the docker password variable to token (#4924) * fix: changes the docker password to token * fix: update docker pass for main release workflow * fix: change DOCKER_PASSWORD to token * null action result when it is empty * Temporarily switch to `ubuntu-20.04` image (#4927) * Ignore ParityResult.IsEmpty in JSON * Fix remove empty results * Merge 1.14.7 * fix licensing header * fix whitespace * Final touches * Make ITraceSerializer generic * refactor * Adds IsTracingFees docs * FIx issue when Result was added to json, because it was created by default on deserialization, even if not serialized * fix shutdown if TraceStorePlugin not Enabled * Review fixes * Fix tests * fix shitespace Co-authored-by: Ruben Buniatyan <rubo@users.noreply.github.com> Co-authored-by: Kamil Chodoła <43241881+kamilchodola@users.noreply.github.com> Co-authored-by: Kamil Chodoła <kamil@nethermind.io> Co-authored-by: Jorge Mederos <46798594+jmederosalvarado@users.noreply.github.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: LukaszRozmej <LukaszRozmej@users.noreply.github.com> Co-authored-by: Oleg Jakushkin <oleg.jakushkin@gmail.com> Co-authored-by: Amirul Ashraf <asdacap@gmail.com> Co-authored-by: MarekM25 <MarekM25@users.noreply.github.com> Co-authored-by: Marek Moraczyński <marekm2504@gmail.com> Co-authored-by: Falco <1364936+FalcoXYZ@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes release workflow environment variable definition.
Changes:
Moved environment variable definition from the step to the job
Types of changes
What types of changes does your code introduce?
Testing
Requires testing
In case you checked yes, did you write tests??