-
Notifications
You must be signed in to change notification settings - Fork 382
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
#5726: use pypi instead of github #5797
Conversation
Thanks Grant this looks like it solves the basic problem. However I had to make the following changes to the readme:
and even then I was getting this error:
let's talk about it. thx. |
I have tested many times and have not been able to reproduce that error with the I have updated the README according to your comment and cleaned things up a bit. I am now unable to get
The post-link just runs It's odd because nothing has changed except that I created a new conda environment based on the current README. So, I think we should merge this soon and maybe update the README to favor the pip method vs. conda method. |
i tried again and now i get this error:
|
Ah, I've seen this! This is usually caused by upgrading to a newer major version of Node. Try rebuilding node-sass ( |
sorry, my mistake! i forgot to clean out my repo. 🐬 |
* origin/master: (21 commits) better source linking, fix missing rel instruction bump version to 0.2.4 bump version to 0.2.3 workaround #5864 fix error setupbase.py on windows (#5863) bump version to 0.2.2, fix #5861 also install our config for the kernelspec manager install our custom css, update author email and website url, add keywords install nbextension, rename entry point for installer, document in readme Leif/fix packaging (#5857) Fix "kernels" cmd (#5849) update version, correct location of version file #5726: use pypi instead of github (#5797) #5810 Fixed lsmagic output. (#5839) #5782 allow setting dpi for png export (#5841) #5909 Fixed Magic import with no arg. (#5840) jarek/5780: reduce duplication in evaluators (#5788) #5767 add links to jvm-repr doc, improve tablesaw doc (#5835) remove "Table Saw" #5597 make widgets publishable (#5830) ... # Conflicts: # beakerx/js/webpack.config.js # beakerx/setup.py
This separate our wrappers (wraps default setuptools commands/tasks with custom commands) into develop (for developers) and install (for users) so that js and java builds are only run for develop and the kernel, kernelspec, and custom_css commands are run for all. With this, we can remove the post-link scripts from the conda recipe (as this serves the same function). This will allow users to install beakerx as follows: