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

FEAT - Split Lab build into an action of its own #30

Merged
merged 33 commits into from
Mar 20, 2023

Conversation

trallard
Copy link
Member

@trallard trallard commented Jan 17, 2023

Closes #27

TODO as part of this PR

  • It should be updated once Enhancements to action #29 is merged into main
  • Add option to use an SHA for Lumino build/link in case this is needed
  • Update comments and docs to match the changes

🎉 I now modified the PR so that we can take the repo and ref of almost any package to build and test with JupyterLab

This run combines jupyterlab/jupyterlab with the branch from mark's lumino's PR that @gabalafou is currently reviewing (scmmmh/lumino) https://github.com/Quansight-Labs/jupyter-a11y-testing/actions/runs/3961294989

@trallard trallard marked this pull request as ready for review January 18, 2023 19:07
@trallard trallard changed the title WIP - FEAT - Split Lab build into an action of its own FEAT - Split Lab build into an action of its own Jan 18, 2023
@trallard trallard requested a review from gabalafou January 18, 2023 19:10
@trallard
Copy link
Member Author

That summary formatting is wonky, so I might need to figure out a way to deal with it... not urgent tho

@gabalafou
Copy link
Contributor

Approved for now. If anything's not working we can fix it in subsequent pull requests.

@trallard trallard self-assigned this Jan 26, 2023
@trallard
Copy link
Member Author

trallard commented Mar 7, 2023

I am going to merge this and if needed work on smaller self-contained improvements

@trallard trallard force-pushed the trallard/install-action branch from 1913661 to 7bd9ea3 Compare March 16, 2023 15:27
@trallard
Copy link
Member Author

Note I had to upgrade to node >= 18 on our deps (ljlab) since now whatever is in master dropped support for node16

Copy link
Contributor

@gabalafou gabalafou left a comment

Choose a reason for hiding this comment

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

I love that we are going to have a way to build a specific version of Lumino into JupyterLab with these changes.

I caught a couple small errors that should probably be fixed before merging.

I also have some comments that shouldn't block this.

trallard and others added 2 commits March 17, 2023 18:20
@trallard
Copy link
Member Author

@gabalafou fixed the issues - except for the jlpm link bit

@trallard trallard merged commit f504604 into main Mar 20, 2023
@trallard trallard deleted the trallard/install-action branch March 20, 2023 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done 💪🏾
Status: Done 💪🏾
Development

Successfully merging this pull request may close these issues.

ENH - Pull JupyterLab build and install steps into reusable action
2 participants