Skip to content

Comments

feat: let test CI run on all release branches#568

Merged
karlfloersch merged 3 commits intomasterfrom
feat/let-ci-run-on-all-release-branches
Apr 23, 2021
Merged

feat: let test CI run on all release branches#568
karlfloersch merged 3 commits intomasterfrom
feat/let-ci-run-on-all-release-branches

Conversation

@karlfloersch
Copy link
Contributor

@karlfloersch karlfloersch commented Apr 22, 2021

Description
Makes the CI run on all branches which start with v indicating that it is a 'version' branch which we can use to prototype breaking changes.

Notes:

@karlfloersch karlfloersch requested a review from tynes April 22, 2021 21:39
@changeset-bot
Copy link

changeset-bot bot commented Apr 22, 2021

⚠️ No Changeset found

Latest commit: 70163bf

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@karlfloersch karlfloersch requested review from gakonst and snario April 22, 2021 21:39
@tynes
Copy link
Contributor

tynes commented Apr 22, 2021

I dont want to bikeshed on this too much but what if it was v0.3.0-rc and the CI runs on *rc? This seems a bit more standard

@snario snario removed their request for review April 22, 2021 23:17
@karlfloersch karlfloersch force-pushed the feat/let-ci-run-on-all-release-branches branch from 54cf894 to 1f89f62 Compare April 23, 2021 21:35
@karlfloersch karlfloersch merged commit 79bffb9 into master Apr 23, 2021
@karlfloersch karlfloersch deleted the feat/let-ci-run-on-all-release-branches branch April 23, 2021 22:07
InoMurko referenced this pull request in omgnetwork/optimism May 25, 2021
* feat: let test CI run on all release branches

* style: *rc

* Add quotes around rc
agusduha pushed a commit that referenced this pull request Sep 25, 2025
theochap pushed a commit that referenced this pull request Dec 10, 2025
theochap pushed a commit that referenced this pull request Jan 15, 2026
Simplifies the deserialization of numeric fields in `OpExecutionPayload`
by directly deserializing them as `U64` values instead of going through
string deserialization with `alloy_serde::quantity::deserialize`.

The following fields are affected:
- `block_number`
- `gas_limit`
- `gas_used`
- `timestamp`
- `blob_gas_used`
- `excess_blob_gas`

This change:
- Removes unnecessary string allocations during deserialization
- Simplifies the code by removing the `IntoDeserializer` dependency
- Aligns with similar fixes made in the main alloy repository

Related to alloy-rs/alloy#2684

Co-authored-by: Claude <noreply@anthropic.com>
emhane pushed a commit that referenced this pull request Feb 4, 2026
Revert all the changes made as part of the hybrid storage experiment.
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.

2 participants