Skip to content

feat: introduce requirements as layers#195

Merged
efiop merged 4 commits intomainfrom
ruslan/infra-1449-featsdk-install_requires
Jan 28, 2026
Merged

feat: introduce requirements as layers#195
efiop merged 4 commits intomainfrom
ruslan/infra-1449-featsdk-install_requires

Conversation

@efiop
Copy link
Contributor

@efiop efiop commented Jan 27, 2026

Drops unused install_requirements that was introduced recently and introduces a new more generic requirements layers. E.g.

requirements = [
    ["foo", "bar"],  # first layer, installed first
    ["baz", "qux"],  # second layer, installed second
    ...
]

For the record: no per-layer cache for now, cached as a whole.

@linear
Copy link

linear bot commented Jan 27, 2026

@efiop efiop force-pushed the ruslan/infra-1449-featsdk-install_requires branch 8 times, most recently from 9e3665a to ac640f0 Compare January 28, 2026 15:52
@efiop efiop force-pushed the ruslan/infra-1449-featsdk-install_requires branch from ac640f0 to 2be1422 Compare January 28, 2026 16:16
Copy link
Member

@chamini2 chamini2 left a comment

Choose a reason for hiding this comment

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

Small detail.

efiop and others added 3 commits January 28, 2026 22:05
Co-authored-by: Matteo Ferrando <matteo.ferrando2@gmail.com>
Co-authored-by: Matteo Ferrando <matteo.ferrando2@gmail.com>
@efiop efiop merged commit ec74c06 into main Jan 28, 2026
9 of 10 checks passed
@efiop efiop deleted the ruslan/infra-1449-featsdk-install_requires branch January 28, 2026 20:54
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