Skip to content

Updated dev install instructions#1622

Merged
Carreau merged 1 commit into
jupyter-server:mainfrom
bloomsa:main
Apr 30, 2026
Merged

Updated dev install instructions#1622
Carreau merged 1 commit into
jupyter-server:mainfrom
bloomsa:main

Conversation

@bloomsa
Copy link
Copy Markdown
Contributor

@bloomsa bloomsa commented Apr 27, 2026

I was going to pick up an issue to work on in the repository and noticed the development installation docs were a little old and could use some freshening up. I chose to add instructions for what I felt were the three most likely tools people would use, Python and pip directly, conda, and uv.

One question I had while working on this... Is node still a dev env requirement? Looking at the package.json it appears to just copy a stylesheet for the static pages? Is there a lighter touch way to set that up or should I retain the mentions and instructions for node?

@Carreau Carreau added this to the 2.18 milestone Apr 30, 2026
@Carreau
Copy link
Copy Markdown
Contributor

Carreau commented Apr 30, 2026

Thanks, I'm not sure cloning your fork is necessary and a few other details, but that's nickpicking and I love seeing activity on this repo.

Merging.

@Carreau Carreau merged commit b389077 into jupyter-server:main Apr 30, 2026
34 of 39 checks passed
@bloomsa
Copy link
Copy Markdown
Contributor Author

bloomsa commented May 1, 2026

I'm open to nitpicks! It felt more "correct" than instructing to clone from jupyter, but removing mention of that altogether makes sense to me

@Carreau
Copy link
Copy Markdown
Contributor

Carreau commented May 1, 2026

Well, it's infinite bike shedding if we discuss details like this; I prefer your evergy to be focused on improving more docs :-)

@bloomsa
Copy link
Copy Markdown
Contributor Author

bloomsa commented May 1, 2026

Ha fair enough. I'm starting on another issue, will tack on some minor cleanup in that one

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.

2 participants