Skip to content

Commit

Permalink
Remove publish feature (#15)
Browse files Browse the repository at this point in the history
The `publish` feature should not be needed anymore after #14.

---------

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
  • Loading branch information
ggwpez authored Dec 22, 2023
1 parent fd32037 commit 970809f
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 9 deletions.
8 changes: 4 additions & 4 deletions .github/workflows/rust.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,16 +30,16 @@ jobs:
run: cargo check --all-targets --no-default-features # still broken --locked

- name: Build
run: cargo build --all-targets --features derive_debug --locked
run: cargo build --all-targets --all-features --locked

- name: Test
run: cargo test --locked --all-targets --features derive_debug
run: cargo test --locked --all-targets --all-features

- name: Clippy
run: cargo clippy --locked --all-targets --features derive_debug -- -D warnings
run: cargo clippy --locked --all-targets --all-features -- -D warnings

- name: Documentation
run: cargo doc --locked --no-deps --features derive_debug
run: cargo doc --locked --no-deps --all-features

- name: Doc Test
run: cargo test --locked --doc
2 changes: 1 addition & 1 deletion Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 6 additions & 2 deletions MAINTAIN.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,12 @@ Info regarding crate maintenance.
The README file of the `proc-macro-warning` crate is not found during normal `cargo publish` invocation. We therefore always publish with the `publish` feature, that adapts the path for publishing.

```bash
# Smoke screen check semver:
cargo semver-checks -p proc-macro-warning
# Replace the version here:
git tag -s -a v1.0.1 -m "Version 1.0.1"
# Check that it works
cargo publish -p proc-macro-warning --features publish --dry-run
cargo publish -p proc-macro-warning --dry-run
# Actually do the publish
cargo publish -p proc-macro-warning --features publish
cargo publish -p proc-macro-warning
```
3 changes: 1 addition & 2 deletions proc-macro-warning/Cargo.toml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
[package]
name = "proc-macro-warning"
version = "1.0.1"
version = "1.0.2"
edition = "2021"
license = "GPL-3.0 OR Apache-2.0"
authors = ["Oliver Tale-Yazdi <oliver@tasty.limo>"]
Expand All @@ -20,4 +20,3 @@ derive = { path = "../ui-tests/derive" }
default = ["derive_debug"]

derive_debug = []
publish = []

0 comments on commit 970809f

Please sign in to comment.