Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Update to a newer version of our crystal flake #156

Merged
merged 1 commit into from
Mar 28, 2024
Merged

Conversation

will
Copy link
Collaborator

@will will commented Mar 28, 2024

The main nixpkgs repo is doing a better job of staying up-to-date with crystal releases, so I've removed most of what we were doing to build our own crystal compiler, and drop down to just some quality-of-life wrapping with common libraries.

Also now updating cb to use newer versions can happen by only updating the nixpkgs reference here, since the crunchydata/nixpkgs flake can be told to follow that input.

@will will requested a review from a team as a code owner March 28, 2024 10:56
The main nixpkgs repo is doing a better job of staying up-to-date with
crystal releases, so I've removed most of what we were doing to build
our own crystal compiler, and drop down to just some quality-of-life
wrapping with common libraries.

Also now updating cb to use newer versions can happen by only updating
the nixpkgs reference here, since the crunchydata/nixpkgs flake can be
told to follow that input.
@@ -3,6 +3,10 @@
# The point is for the user to remove these configuration records
# one by one as the reported problems are removed from the code base.

# Seems broken in current (v1.6.1) build of ameba, so disabling
Copy link
Member

Choose a reason for hiding this comment

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

👍

@will will merged commit cafe854 into main Mar 28, 2024
2 checks passed
@will will deleted the simplercrystalflake branch March 28, 2024 17:09
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