Skip to content

fix: replace facet's constant by const_get - #570

Open
doudou wants to merge 1 commit into
transition-to-runkitfrom
remove_facet_constant
Open

doudou wants to merge 1 commit into
transition-to-runkitfrom
remove_facet_constant

Conversation

@doudou

@doudou doudou commented Sep 17, 2026

Copy link
Copy Markdown
Member

No description provided.

@chhtz chhtz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. I guess we didn't hit this, because it is not used during import or build-phase. (N.B., we still check out syskit@master, which appears to be stale ...)

Nevertheless, there are still calls to constant left in other files, e.g.

@doudou

doudou commented Sep 21, 2026

Copy link
Copy Markdown
Member Author

we still check out syskit@master, which appears to be stale

Very stale. I sent a message beginning of the year that we would merge the transition-to-runkit branch back into master and that people should track the v2.1.0 tag (which is the current master)

@doudou
doudou force-pushed the remove_facet_constant branch from d464ada to 8382689 Compare September 21, 2026 14:41
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