Skip to content
This repository was archived by the owner on Nov 19, 2021. It is now read-only.

bump usage of techdocs common package to latest version#169

Merged
ottosichert merged 11 commits intomainfrom
emmai/bump-techdocs-common
Oct 13, 2021
Merged

bump usage of techdocs common package to latest version#169
ottosichert merged 11 commits intomainfrom
emmai/bump-techdocs-common

Conversation

@emmaindal
Copy link
Copy Markdown
Member

@emmaindal emmaindal commented Oct 7, 2021

closes: #92

This PR upgrades the techdocs common page to the latest version. Also falls back to the default docker image version if no docker image argument is passed through when running the command.

Signed-off-by: Emma Indal <emmai@spotify.com>
Signed-off-by: Emma Indal <emmai@spotify.com>
Signed-off-by: Emma Indal <emmai@spotify.com>
@emmaindal emmaindal requested a review from a team as a code owner October 7, 2021 09:37
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Oct 7, 2021

🦋 Changeset detected

Latest commit: 073df46

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@techdocs/cli Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@emmaindal
Copy link
Copy Markdown
Member Author

Looking into why the e2e test doesn't pass, as all the commands works without any probs when running locally.

@emmaindal
Copy link
Copy Markdown
Member Author

emmaindal commented Oct 7, 2021

Looks like there is a problem with processes not being killed, as I get the error [Errno 48] Address already in use. So I'll look into this more!

Edit: there was an additional problem too. I introduced the techdocs-core plugin which is not being installed so that was why it failed before, so installing techdocs-core now in the workflow.

Comment thread packages/techdocs-cli/src/commands/generate/generate.ts Outdated
@ottosichert ottosichert merged commit 14d64b4 into main Oct 13, 2021
@ottosichert ottosichert deleted the emmai/bump-techdocs-common branch October 13, 2021 10:54
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.

techdocs serve does not work

5 participants