Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Cleanup flake #185

Merged
merged 2 commits into from
May 24, 2023
Merged

Cleanup flake #185

merged 2 commits into from
May 24, 2023

Conversation

nikstur
Copy link
Member

@nikstur nikstur commented May 23, 2023

Add a proper description to flake.

Use nativeCheckInputs instead of checkInputs because it is more
semantically correct even if checkInputs works with Crane.

nikstur added 2 commits May 24, 2023 01:08
@nikstur
flake: add proper description
8e4de78
@nikstur
flake: checkInputs -> nativeCheckInputs
740f7f9 
Use nativeCheckInputs instead of checkInputs because it is more
semantically correct even if checkInputs works with Crane.
@nikstur nikstur requested review from blitz and RaitoBezarius May 23, 2023 23:09
RaitoBezarius
RaitoBezarius approved these changes May 23, 2023
View reviewed changes
Copy link
Member

@RaitoBezarius RaitoBezarius left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

@nikstur nikstur merged commit dc52f03 into master May 24, 2023
@nikstur nikstur deleted the cleanup-flake branch July 20, 2023 22:02
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@RaitoBezarius RaitoBezarius RaitoBezarius approved these changes

@blitz blitz Awaiting requested review from blitz

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nikstur @RaitoBezarius