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 example in haskell-use-cases.rst to rules_oci #2137

Merged
merged 1 commit into from
Mar 21, 2024

Conversation

iteratee
Copy link
Contributor

rules_docker is not just deprecated, the repository is archived and read only. Change the example in the use cases doc to be a rules_oci example.

Missing from the example is build-timestamp stamping. That isn't sufficient reason to not update the example with the current state of rules_docker.

rules_docker is not just deprecated, the repository is archived and read
only. Change the example in the use cases doc to be a rules_oci example.

Missing from the example is build-timestamp stamping. That isn't
sufficient reason to not update the example with the current state of
rules_docker.
@iteratee iteratee requested a review from avdv as a code owner March 20, 2024 18:50

*n.b.* Due to the `current inability`_ of Nix to be used on macOS (darwin) for building Docker images, it's currently
not possible to build Docker images for Haskell binaries as above using ``rules_docker`` and Nixpkgs on macOS.
not possible to build Docker images for Haskell binaries as above using ``rules_oci`` and Nixpkgs on macOS.
Copy link
Member

Choose a reason for hiding this comment

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

Since the referenced nixpgks issue is already resolved, we should re-assess the situation now and probably remove that paragraph.

Not an issue with this PR, though. I have created a new issue to track this: #2138

Copy link
Member

@avdv avdv left a comment

Choose a reason for hiding this comment

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

Looks good, thank you @iteratee !

@avdv avdv added the merge-queue merge on green CI label Mar 21, 2024
@mergify mergify bot merged commit bf2e6cd into tweag:master Mar 21, 2024
49 checks passed
@mergify mergify bot removed the merge-queue merge on green CI label Mar 21, 2024
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