Uh oh!
There was an error while loading. Please reload this page.
Add kani logo - #1157
Merged
Merged
Conversation
danielsnforce-pushed
the
logo
branch
2 times, most recently
from
May 4, 2022 14:51
228a75b to
7a13a1dComparejaisnan
approved these changes
May 4, 2022
feliperodri added a commit
to feliperodri/kani
that referenced
this pull request
Aug 17, 2026
Added alongside the logo in model-checking#1157 but never referenced. The documentation site is built by `scripts/build-docs.sh` and published from `docs/book/`, so a repository-root icon never reaches it; mdBook takes its favicon from `theme/favicon.{png,svg}`, which this repository does not override. It is also absent from the `include` list in `Cargo.toml`, so it is not packaged into the `kani-verifier` crate, and GitHub does not use repository-root icons.
feliperodri added a commit
to tautschnig/kani
that referenced
this pull request
Aug 25, 2026
…el-checking#4741) Housekeeping for a few files that are either stale or undocumented, plus proper citation metadata now that the Kani paper is published. Each commit stands alone. **Citation metadata.** Adds `CITATION.cff` whose `preferred-citation` points at the ASE '26 paper, so GitHub's *Cite this repository* button emits the paper rather than a bare software reference, and adds a "Citing Kani" section to the README with the ACM Reference Format and a ready-to-paste BibTeX entry. **Removes the VSTTE 2024 paper sources.** Nothing builds them, the paper is published, and the ASE 2026 paper is now the citation of record. Still available in git history. **Removes `favicon.ico`.** Added with the logo in model-checking#1157 and never referenced. The docs site is published from `docs/book/`, so a repository-root icon never reaches it, and mdBook takes its favicon from `theme/favicon.{png,svg}`, which we don't override. It's also absent from the `include` list in `Cargo.toml`, so it isn't packaged into the `kani-verifier` crate. **Removes six orphaned `Dockerfile.bundle-test-*` files.** They were built by the `TestAlternativePlatforms` job of `release.yml`, removed in model-checking#3744 when Ubuntu 18.04/AL2 support was dropped; that PR deleted only the 18.04 Dockerfile and left the rest behind. They're also stale — `bundle-test-al2` installs `kani-verifier` from crates.io with the bundle-install lines commented out, so it wasn't testing the bundle under test. Bundle installation is covered natively by the `TestBundle` job across the macOS and Ubuntu runners. **Documents the remaining CI helpers.** Adds `scripts/ci/README.md` explaining that `Dockerfile.bundle-release-24-04` builds the `ghcr.io/model-checking/kani-ubuntu-24.04` image published by the `Package Docker` job on release tags, why it can't be built from a clean checkout, and the commands that produce the artifacts it expects. Also records why the root `.dockerignore` exists and that `run-copyright-check.sh` needs GNU `xargs`. Manual testing: - `CITATION.cff` validates against CFF schema 1.2.0 (`cffconvert --validate`), and `ruby-cff` (the library behind GitHub's citation widget) renders it as a complete `@inproceedings` entry with all authors and the DOI. - Ran the copyright check over all tracked files, including the new `CITATION.cff`: passes. Note it needs a BSD-compatible `xargs` invocation to run on macOS. - Grepped the tree for references to every deleted path: none remain. By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of changes:
Adds our logo and favicon to the repo.
Resolved issues:
NA
Call-outs:
Testing:
How is this change tested? NA
Is this a refactor change? NA
Checklist
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.