Use cargo metadata to collect information about packages (#600)

Added support of `{ workspace = true }` syntax.

Author: xgreenx

.github/workflows/test.yml

@@ -44,7 +44,7 @@ jobs:
           check-repo: ${{ github.event_name == 'push' }}
 
   publish: # publish dependency-free release with only self-contained dist directory
-    if: github.repository == 'katyo/publish-crates' && github.event_name == 'push' && github.ref_name == 'main'
+    if: github.event_name == 'push' && github.ref_name == 'main'
     runs-on: ubuntu-latest
     needs:
       - build

README.md

@@ -3,6 +3,9 @@
 
 # Publish Rust crates using GitHub Actions
 
+The action is using [`cargo metadata`](https://doc.rust-lang.org/cargo/commands/cargo-metadata.html) with format version 
+`1` to collect the information about crates and workspace.
+
 ## Features
 
 - Reads manifests to get info about crates and dependencies
@@ -13,6 +16,8 @@
 - Publishes updated crates in right order according to dependencies
 - Awaits when published crate will be available in registry before publishing crates which depends from it
 - Works fine in workspaces without cyclic dependencies
+- Support `{ workspace = true }` syntax in the `Cargo.toml`. [This](https://rust-lang.github.io/rfcs/2906-cargo-workspace-deduplicate.html)
+  feature was stabilized in Rust 1.64.
 
 ## Unimplemented features
 

__tests__/Cargo.toml

@@ -1,19 +1,31 @@
 [workspace]
 members = [
   "pkg-sys",
+  "pkg-skip",
   "pkg-build",
   "pkg-lib",
   "pkg-bin",
   'pkg-dev',
 ]
 
+[workspace.package]
+version = "0.1.0"
+
 [package]
 name = "pkg-all"
 version = "0.1.0"
 
 [dependencies]
 pkg-lib = { version = "0.1.0", path = "./pkg-lib" }
+subcrate-d = { workspace = true, path = "../workspace/subcrate_d" }
+subcrate-e = { workspace = true, path = "../workspace/subcrate_e" }
+subcrate-f = { workspace = true, path = "../workspace/subcrate_f" }
 
 [dependencies.pkg-bin]
 version = "0.1.0"
 path = "./pkg-bin"
+
+[workspace.dependencies]
+subcrate-d = { version = "0.1.0", path = "./workspace/subcrate_d" }
+subcrate-e = { version = "0.1.0", path = "./workspace/subcrate_e" }
+subcrate-f = { version = "0.1.0", path = "./workspace/subcrate_f" }

__tests__/main.test.ts

@@ -3,22 +3,22 @@ import {findPackages, checkPackages, sortPackages} from '../src/package'
 import {githubHandle, lastCommitDate} from '../src/github'
 import {semver} from '../src/utils'
 import {join} from 'path'
-import {exec} from '@actions/exec'
 const pkg_dir = __dirname
 
 test('find packages', async () => {
     const packages = await findPackages(pkg_dir)
 
-    expect(Object.keys(packages).length).toBe(6)
+    expect(Object.keys(packages).length).toBe(9)
 
     const pkg_all = packages['pkg-all']
     const pkg_sys = packages['pkg-sys']
     const pkg_lib = packages['pkg-lib']
     const pkg_bin = packages['pkg-bin']
+    const subcrate_e = packages['subcrate-e']
 
     expect(pkg_all.path).toBe(pkg_dir)
     expect(pkg_all.version).toBe('0.1.0')
-    expect(Object.keys(pkg_all.dependencies).length).toBe(2)
+    expect(Object.keys(pkg_all.dependencies).length).toBe(5)
 
     expect(pkg_sys.path).toBe(join(pkg_dir, 'pkg-sys'))
     expect(pkg_sys.version).toBe('0.1.0')
@@ -31,6 +31,10 @@ test('find packages', async () => {
     expect(pkg_bin.path).toBe(join(pkg_dir, 'pkg-bin'))
     expect(pkg_bin.version).toBe('0.1.0')
     expect(Object.keys(pkg_bin.dependencies).length).toBe(3)
+
+    expect(subcrate_e.path).toBe(join(pkg_dir, 'workspace/subcrate_e'))
+    expect(subcrate_e.version).toBe('0.1.0')
+    expect(Object.keys(subcrate_e.dependencies).length).toBe(1)
 })
 
 test('check packages', async () => {
@@ -46,6 +50,9 @@ test('sort packages', async () => {
         'pkg-sys',
         'pkg-lib',
         'pkg-build',
+        'subcrate-d',
+        'subcrate-f',
+        'subcrate-e',
         'pkg-dev',
         'pkg-bin',
         'pkg-all'

__tests__/pkg-lib/Cargo.toml

@@ -9,6 +9,9 @@ lazy_static = "1.0"
 version = "0.1.0"
 path = "../pkg-sys"
 
+[dev-dependencies]
+lazy_static = "1.0"
+
 # test that local cyclic dev-deps are allowed using paths
 [dev-dependencies.pkg-dev]
 path = "../pkg-dev"

__tests__/pkg-skip/Cargo.toml

@@ -0,0 +1,12 @@
+[package]
+name = "pkg-skip"
+version = "0.1.0"
+publish = false
+
+[dependencies]
+pkg-lib = { version = "0.1.0", path = "../pkg-lib" }
+clap = "^2"
+
+[dev-dependencies.pkg-dev]
+version = "0.1.0"
+path = "../pkg-dev"

__tests__/pkg-skip/src/main.rs

@@ -0,0 +1 @@
+fn main() {}

__tests__/workspace/subcrate_d/Cargo.toml

@@ -0,0 +1,6 @@
+[package]
+name = "subcrate-d"
+version = { workspace = true }
+
+[dev-dependencies.subcrate-f]
+path = "../subcrate_f"

__tests__/workspace/subcrate_d/src/lib.rs

@@ -0,0 +1,10 @@
+pub const TEST_VALUE: u8 = 5u8;
+
+#[cfg(test)]
+mod tests {
+    #[test]
+    fn it_works() {
+        let result = 2 + 2;
+        assert_eq!(result, 4);
+    }
+}

__tests__/workspace/subcrate_e/Cargo.toml

@@ -0,0 +1,10 @@
+[package]
+name = "subcrate-e"
+version = { workspace = true }
+
+[dependencies.subcrate-d]
+workspace = true
+path = "../subcrate_d"
+
+[dev-dependencies.subcrate-f]
+path = "../subcrate_f"