Skip to content

Commit

Permalink
fix: Remove publish check.
Browse files Browse the repository at this point in the history
  • Loading branch information
milesj committed Nov 5, 2023
1 parent 87c5c78 commit 49599d3
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 9 deletions.
1 change: 0 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,6 @@ jobs:
Packages to be built and published must have the following configuration in their `Cargo.toml`:

- The `package.publish` setting should be omitted or set to `true`.
- The `lib.crate-type` setting should be set to `cdylib`.

```toml
Expand Down
7 changes: 0 additions & 7 deletions index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,6 @@ async function findBuildablePackages() {
}

interface Manifest {
package?: { publish?: boolean };
profile?: Record<string, { 'opt-level'?: string }>;
}

Expand All @@ -105,12 +104,6 @@ async function findBuildablePackages() {
core.info(`Found ${pkg.name}, loading manifest ${pkg.manifest_path}, checking targets`);

const manifest = TOML.parse(fs.readFileSync(pkg.manifest_path, 'utf8')) as Manifest;
const publish = manifest.package?.publish ?? true;

if (!publish) {
core.info(`Skipping ${pkg.name}, not publishable`);
return;
}

pkg.targets.forEach((target) => {
if (target.crate_types.includes('cdylib')) {
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@moonrepo/build-proto-plugin",
"version": "0.1.0",
"version": "0.1.1",
"description": "A GitHub action to build, optimize, and prepare proto WASM plugins for release.",
"main": "dist/index.js",
"scripts": {
Expand Down

0 comments on commit 49599d3

Please sign in to comment.