Skip to content

[CRE][Settings] Switch feature flag type to int64#1867

Merged
bolekk merged 2 commits intomainfrom
flag_change_type
Mar 2, 2026
Merged

[CRE][Settings] Switch feature flag type to int64#1867
bolekk merged 2 commits intomainfrom
flag_change_type

Conversation

@bolekk
Copy link
Contributor

@bolekk bolekk commented Feb 28, 2026

[CRE][Settings] Switch feature flag type to int64

@bolekk bolekk requested a review from a team as a code owner February 28, 2026 01:45
@github-actions
Copy link

👋 bolekk, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

@github-actions
Copy link

github-actions bot commented Feb 28, 2026

⚠️ API Diff Results - github.com/smartcontractkit/chainlink-common

⚠️ Breaking Changes (1)

pkg/settings (1)
  • Time — Type changed:
func(
  time.Time
)
- Setting[time.Time]
+ Setting[github.com/smartcontractkit/chainlink-common/pkg/config.Timestamp]

✅ Compatible Changes (2)

pkg/config (2)
  • ParseTimestamp — ➕ Added

  • Timestamp — ➕ Added


📄 View full apidiff report

@bolekk bolekk marked this pull request as draft February 28, 2026 02:08
@bolekk bolekk marked this pull request as ready for review March 2, 2026 03:01
@bolekk bolekk enabled auto-merge March 2, 2026 17:18
@bolekk bolekk added this pull request to the merge queue Mar 2, 2026
Merged via the queue into main with commit 40eba75 Mar 2, 2026
36 of 37 checks passed
@bolekk bolekk deleted the flag_change_type branch March 2, 2026 17:34
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