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.

Sign up for GitHub

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

github: create actions to publish crates #578

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

github: create actions to publish crates #578

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
fa0185a
cargo: set specific versions for each dependency
nitnelave May 12, 2023
1377d5a
github: create actions to publish crates
nitnelave May 12, 2023
e1aa2bf
cargo: depend on the published version of lldap_auth
nitnelave May 12, 2023
1b2dfbe
Publish job using matrix
martadinata666 May 12, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Publish job using matrix 
Change publish job with matrix, don't fail if another job fail.
  • Loading branch information
@martadinata666
martadinata666 authored May 12, 2023
commit 1b2dfbe52ef83628ab8c06f09101ab76b94307d8
41 changes: 7 additions & 34 deletions .github/workflows/rust.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -117,50 +117,23 @@ jobs:
name: Publish on crates.io
if: ${{ needs.pre_job.outputs.should_skip != 'true' || (github.event_name == 'push' && github.ref == 'refs/heads/main') || github.event_name == 'release' }}
needs: pre_job
strategy:
fail-fast: false
matrix:
target: [lldap_auth, lldap, lldap_app, lldap_set_password, lldap_migration_tool]
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3.5.2
- uses: actions-rs/toolchain@v1
Copy link
Member Author

Choose a reason for hiding this comment

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

Note: we might want (in a future PR) to migrate our rust installation to this action instead.

Copy link
Contributor

Choose a reason for hiding this comment

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

actually you probably don't want to use actions-rs because its unmaintained

with:
toolchain: stable
override: true
- name: Publish lldap_auth crate
- name: Publish ${{ matrix.target }} crate
uses: katyo/publish-crates@v2
with:
args: -p lldap_auth
dry-run: ${{ github.event_name != 'release' }}
check-repo: ${{ github.event_name != 'pull_request' }}
registry-token: ${{ secrets.CARGO_REGISTRY_TOKEN }}
ignore-unpublished-changes: ${{ github.event_name != 'release' }}
- name: Publish lldap crate
uses: katyo/publish-crates@v2
with:
args: -p lldap
dry-run: ${{ github.event_name != 'release' }}
check-repo: ${{ github.event_name != 'pull_request' }}
registry-token: ${{ secrets.CARGO_REGISTRY_TOKEN }}
ignore-unpublished-changes: ${{ github.event_name != 'release' }}
- name: Publish lldap_app crate
uses: katyo/publish-crates@v2
with:
args: -p lldap_app
dry-run: ${{ github.event_name != 'release' }}
check-repo: ${{ github.event_name != 'pull_request' }}
registry-token: ${{ secrets.CARGO_REGISTRY_TOKEN }}
ignore-unpublished-changes: ${{ github.event_name != 'release' }}
- name: Publish lldap_migration_tool crate
uses: katyo/publish-crates@v2
with:
args: -p lldap_migration_tool
dry-run: ${{ github.event_name != 'release' }}
check-repo: ${{ github.event_name != 'pull_request' }}
registry-token: ${{ secrets.CARGO_REGISTRY_TOKEN }}
ignore-unpublished-changes: ${{ github.event_name != 'release' }}
- name: Publish lldap_set_password crate
uses: katyo/publish-crates@v2
with:
args: -p lldap_set_password
args: -p ${{ matrix.target }}
dry-run: ${{ github.event_name != 'release' }}
check-repo: ${{ github.event_name != 'pull_request' }}
registry-token: ${{ secrets.CARGO_REGISTRY_TOKEN }}
ignore-unpublished-changes: ${{ github.event_name != 'release' }}