Skip to content

more ci updates#91

Merged
nyurik merged 3 commits intomainfrom
ci
Jun 3, 2025
Merged

more ci updates#91
nyurik merged 3 commits intomainfrom
ci

Conversation

@nyurik
Copy link
Owner

@nyurik nyurik commented Jun 3, 2025

No description provided.

Copilot AI review requested due to automatic review settings June 3, 2025 02:57
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates CI configurations and build scripts by refining version checking and documentation generation steps in the justfile and adjusting the dependabot workflow token syntax.

  • Updated the check-if-published recipe in justfile to use a package parameter and streamlined version output messages.
  • Modified the docs and test-doc recipes for enhanced documentation build control.
  • Adjusted the dependabot workflow configuration for token interpolation.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
justfile Updated check-if-published, docs, and test-doc recipes to improve version detection and documentation generation.
.github/workflows/dependabot.yml Minor syntax update for the github-token value in the Dependabot workflow.
Comments suppressed due to low confidence (1)

.github/workflows/dependabot.yml:15

  • [nitpick] Review the removal of quotes around the secret; while this may work as intended, using quotes can help prevent potential parsing issues in some YAML contexts. Ensure this change is consistent with other workflow files.
github-token: ${{secrets.GITHUB_TOKEN}}

check-if-published: (assert-cmd 'jq')
check-if-published package=main_crate: (assert-cmd 'jq')
#!/usr/bin/env bash
set -euo pipefail
Copy link

Copilot AI Jun 3, 2025

Choose a reason for hiding this comment

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

The new check-if-published command removes logging of the crate name, which could be useful for debugging version mismatch issues. Consider reintroducing a log for the package name to improve traceability.

Suggested change
set -euo pipefail
set -euo pipefail
echo "Checking crate: '{{package}}'"

Copilot uses AI. Check for mistakes.
@codecov
Copy link

codecov bot commented Jun 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.13%. Comparing base (01417f6) to head (1ef5a18).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #91   +/-   ##
=======================================
  Coverage   85.13%   85.13%           
=======================================
  Files          13       13           
  Lines         222      222           
  Branches      222      222           
=======================================
  Hits          189      189           
  Misses          7        7           
  Partials       26       26           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@nyurik nyurik merged commit 6fc31e0 into main Jun 3, 2025
14 checks passed
@nyurik nyurik deleted the ci branch June 3, 2025 03:35
@github-actions github-actions bot mentioned this pull request Jun 6, 2025
nyurik added a commit that referenced this pull request Jun 6, 2025
## 🤖 New release

* `sqlite-hashes`: 0.10.1 -> 0.10.2 (✓ API compatible changes)

<details><summary><i><b>Changelog</b></i></summary><p>

<blockquote>

##
[0.10.2](v0.10.1...v0.10.2)
- 2025-06-06

### Fixed

- fix justfile jq assertion
- fix justfile jq test

### Other

- set release-plz CI
([#94](#94))
- validate if published in CI
([#93](#93))
- ci updates ([#92](#92))
- more ci updates
([#91](#91))
- More rework of CI and justfiles
([#90](#90))
- update dependabot
- ci cleanup
</blockquote>


</p></details>

---
This PR was generated with
[release-plz](https://github.com/release-plz/release-plz/).

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Yuri Astrakhan <yuriastrakhan@gmail.com>
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.

1 participant