Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[legacy-framework] Move display package to nextjs fork #2989

Merged
merged 16 commits into from
Dec 3, 2021

Conversation

beerose
Copy link
Contributor

@beerose beerose commented Nov 19, 2021

Part of blitz-js/legacy-framework#391

I haven't modified or deleted display package functions yet.

Bug Checklist

  • Integration test added (see test docs if needed)

Feature Checklist

kodiakhq[bot]
kodiakhq bot previously approved these changes Nov 19, 2021
kodiakhq[bot]
kodiakhq bot previously approved these changes Nov 19, 2021
@beerose beerose marked this pull request as draft November 19, 2021 15:23
@beerose beerose marked this pull request as ready for review November 22, 2021 15:38
kodiakhq[bot]
kodiakhq bot previously approved these changes Nov 22, 2021
imcodingideas
imcodingideas previously approved these changes Nov 27, 2021
Copy link

@imcodingideas imcodingideas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@@ -111,6 +112,7 @@
"node-libs-browser": "^2.2.1",
"npm-which": "^3.0.1",
"null-loader": "4.0.1",
"ora": "^5.3.0",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably can import from next/dist/compiled/ora

Comment on lines 86 to 88
const withWarning = (str: string) => {
return `⚠️ ${c.yellow(str)}`
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove in favor of tslog.warn

Comment on lines 98 to 100
const withX = (str: string) => {
return `${c.red.bold('✕')} ${str}`
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove in favor of tslog.error

Comment on lines 139 to 141
const warning = (msg: string) => {
console.log(withCaret(withWarning(msg)))
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove in favor of tslog.warn

@beerose beerose dismissed stale reviews from imcodingideas and kodiakhq[bot] via f0fcad0 December 1, 2021 13:16
kodiakhq[bot]
kodiakhq bot previously approved these changes Dec 1, 2021
kodiakhq[bot]
kodiakhq bot previously approved these changes Dec 1, 2021
kodiakhq[bot]
kodiakhq bot previously approved these changes Dec 1, 2021
kodiakhq[bot]
kodiakhq bot previously approved these changes Dec 1, 2021
@flybayer flybayer merged commit 5460fbb into canary Dec 3, 2021
@flybayer flybayer deleted the move-display-package branch December 3, 2021 21:46
@itsdillon itsdillon changed the title Move display package to nextjs fork [legacy-framework] Move display package to nextjs fork Jul 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants