-
Notifications
You must be signed in to change notification settings - Fork 885
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
Install packages that are hosted on github #36
Comments
We currently only support importing from NPM, but we do have (basic) import from github working in local builds. However, it's gonna require a lot of testing & love to ensure it plays nice with our dep resolver, which is important bc we def don't want dep mismatches throwing errors in your console that you normally wouldn't get locally. (PS - I just saw/responded to your tweet about this! 🍻) |
Understood. FYI. Plz consider this an important feature: developers will use StackBlitz to test and demo... often with libraries recently fixed (e.g. nightly builds).
|
@EricSimons - If you guys need extra help with StackBlitz, ping me (I would love to get involved). |
Excellent point, I hadn't even considered this. I'm gonna bump the priority on this feature to ensure it's in prod by EOM 👍 Def would love to have you get involved with SB! I'm putting together a list of things we could use help with actually, hoping to get it posted on the repo this week. Also just followed you on twitter so we can DM :) |
@EricSimons - ping me on email (github profile), let me when/how I can help. Cheers. |
@EricSimons - I am still blocked with this. I would like to help resolve this one!
|
@ThomasBurleson so sorry about the delay here! Past month or so has been insane 😓 I'm going to start working on this today and will have it out early tmrw. Will post back here once it's live! |
@EricSimons - wonderful news. Consider me a fan and a resource ! |
@ThomasBurleson it's live on prod! You can simply enter This is definitely a beta feature so lmk if you run into any issues! There are also some limitations right now (it won't resolve nested sub-packages in the github repo, etc) but lmk if that's a req for you and I'll prioritize some time to put in a fix for it. PS - we're still getting our ducks in a row for the next week or two, but keep an eye out for an email from me reg getting involved w/ stackblitz :) |
@EricSimons - I so admire professionalism and communication. Kudos to you for both. I will try this beta feature today. Thank you. |
@EricSimons - Your fix(es) work wonderfully. Here is a blitz with the latest code in our nightly build: https://stackblitz.com/edit/angular-pu8pc6?file=app%2Ftest.component.ts Closing as fixed! |
@ThomasBurleson No problem at all! Thanks for the kind words :) It's great to see everything is working well for the flex-layout demo. Btw I put the final touches on the resolution logic today (per #149 (comment)), so it's now possible to import SASS/CSS, nested packages, etc from github repos as well (here's a demo of the latest GH build of Ang Material w/ a custom theme: https://stackblitz.com/edit/angular-m9axcs?file=theme.scss) |
@EricSimons - So I get the |
@EricSimons this feature does not appear to be working. Here's a screenshot of the project you referenced: https://stackblitz.com/edit/angular-m9axcs?file=theme.scss |
Hi this would be a very useful feature, is there any news on this? |
Tagging as related to #1195 |
I want to use the nightly build of
@angular/flex-layout
.Normally I can do one of these:
npm -i github.com/angular/flex-layout-builds
, ornpm -i angular/flex-layout-builds
None of these seem to work in the StackBlizt IDE.
The text was updated successfully, but these errors were encountered: