GitLab

I opened an issue about it as it’s failing in master also

This is a 🔦 documentation change

so you don't have to search for them

* Typo: run -> runs

* fix accidental commit new line

ahh I messed up the last commit when editing from the github UI

* Add tests for a full `bankai build`

* Fix lint

* add some logs to check appveyor output

* abababababababb

* angery

* Update split-require to fix Windows build

* read that dir?

* Revert "read that dir?"

This reverts commit b1a1eeb96495d175cf88bd5d2ad956fba429f963.

<!--
Thanks for creating a Pull Request 😄 ! Before you submit, please read the following:
- Read our CONTRIBUTING.md file before submitting a patch.
- By making a contribution, you agree to our Developer Certificate of Origin.
-->

This is a 🙋 feature

<!-- Provide a general summary of the changes in the title above -->

This implements the 2nd panel for `bankai start`, showing console output from the server rendered app (don't know if there's anything else?).

SSR uses a separate Console instance, injected as a global `console` variable using [require-with-global](https://github.com/goto-bus-stop/require-with-global). That `console` writes to a stream which is saved by lib/ui.js. Pressing 2 in the `bankai start` TUI will switch to the console panel, which is an [ansi-scrollbox](https://github.com/goto-bus-stop/ansi-scrollbox) that renders the saved console output. ansi-scrollbox still needs more features and optimization (it's currently probably very slow for many thousands of lines) but it does sort of work.

## Checklist
<!-- Remove items that do not apply. For completed items, change [ ] to [x]. -->
- [ ] tests pass
- [ ] tests and/or benchmarks are included
- [ ] documentation is changed or added

## Semver Changes
Minor

* Use `tmp` module throughout tests

It was already being used in some places, and is a bit less verbose.

* Create ./tmp before running tests.

* Remove unused dependency

* debug

* is it less flakey with an unreleased version of tmp?

This is a 🐛 

When I was trying to add the faviconTag to the document I found this bug. I created a `faviconNode`, called `createEdge()` inside it, then the graph's change event was emited, then it [emits the compiler's change event](https://github.com/choojs/bankai/blob/master/index.js#L56) immediately, but at this moment, [compiler's change event](https://github.com/choojs/bankai/blob/master/lib/cmd-build.js#L47) has not yet been listened.

<!-- Provide a general summary of the changes in the title above -->

## Checklist
<!-- Remove items that do not apply. For completed items, change [ ] to [x]. -->
- [x] tests pass
- [ ] tests and/or benchmarks are included
- [ ] documentation is changed or added

## Context
No

## Semver Changes
minor

## Checklist
<!-- Remove items that do not apply. For completed items, change [ ] to [x]. -->
- [x] tests pass

## Semver Changes
Patch

This change swaps out `open` with `opn` which has a compatible API, but
is cross platform, and maintained.

* use utils.find()

* unify url format