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
Use the local theme as a requirement #4
Conversation
Signed-off-by: Tim 'mithro' Ansell <me@mith.ro>
Signed-off-by: Daniel Lim Wee Soong <weesoong.lim@gmail.com>
8a0a2ae
to
4ce9f80
Compare
Signed-off-by: Daniel Lim Wee Soong <weesoong.lim@gmail.com>
@mithro ping |
This doesn't seem to use the local theme as a requirement? I would expect the |
Signed-off-by: Daniel Lim Wee Soong <weesoong.lim@gmail.com>
Signed-off-by: Daniel Lim Wee Soong <weesoong.lim@gmail.com>
Oh right. I applied the changes in #3 to this branch (renaming |
Signed-off-by: Daniel Lim Wee Soong <weesoong.lim@gmail.com>
Signed-off-by: Daniel Lim Wee Soong <weesoong.lim@gmail.com>
Apparently Checkout the preview. It's our theme now. |
I guess this LGTM, I wonder if we can reduce the change delta to upstream however? |
What do you mean? |
Try to fix the build error that occurs in #2
Edit: Great! The fix works!