Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

fix: make message-port-protocol non dev dependency #3393

Merged
merged 1 commit into from
Nov 13, 2020
Merged

Conversation

Gozala
Copy link
Contributor

@Gozala Gozala commented Nov 13, 2020

Fix message-port-client/server packages by declaring message-port-protocol as dependency that it is, instead of devDependency.

@Gozala Gozala merged commit cea7317 into master Nov 13, 2020
@Gozala Gozala deleted the fix/msg-port-deps branch November 13, 2020 09:45
achingbrain added a commit to ipfs/aegir that referenced this pull request Nov 13, 2020
Adds a convenience flag to only check for missing production deps.

Also fixes the spinner when dep-checks fail so the formatting of the
error message isn't all kinds of messed up.

Would have caught ipfs/js-ipfs#3393
achingbrain added a commit to ipfs/aegir that referenced this pull request Nov 13, 2020
Adds a convenience flag to only check for missing production deps.

Also allows the user to pass files to check instead of always using
the defaults.

Also, also fixes the spinner when dep-checks fail so the formatting of the
error message isn't all kinds of messed up.

Would have caught ipfs/js-ipfs#3393
achingbrain added a commit to ipfs/aegir that referenced this pull request Nov 13, 2020
Adds a convenience flag to only check for missing production deps.

Also allows the user to pass files to check instead of always using
the defaults.

Also, also fixes the spinner when dep-checks fail so the formatting of the
error message isn't all kinds of messed up.

Would have caught ipfs/js-ipfs#3393
achingbrain added a commit to ipfs/aegir that referenced this pull request Nov 16, 2020
Adds a convenience flag to only check for missing production deps.

Also fixes the spinner when dep-checks fail so the formatting of the error message isn't all kinds of messed up.

Also, also adds `--ignore` options to the dep check so we can pass options to aegir and to the underlying dependency-check binary.

Would have caught ipfs/js-ipfs#3393
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants