Skip to content
This repository has been archived by the owner on Feb 16, 2023. It is now read-only.

Mark this repo as archived #158

Open
telamonian opened this issue Nov 19, 2020 · 10 comments
Open

Mark this repo as archived #158

telamonian opened this issue Nov 19, 2020 · 10 comments
Labels
question Further information is requested

Comments

@telamonian
Copy link
Member

Now that the TOC extension is part of jupyterlab core, we should probably mark this repo as archived, and also add some helpful text for new/latecomers (eg "If you have jlab v<x, install this extension. Otherwise, TOC is already built into jlab" or whatever).

Pinging the people who actually worked on this extension:

@kgryte
@marthacryan
@jasongrout
@lresende

@telamonian telamonian added the question Further information is requested label Nov 19, 2020
@kgryte
Copy link
Member

kgryte commented Nov 19, 2020

I haven't kept up with work in core, so I'm not sure if the core implementation has feature parity. There was talk some time ago that this extension may need to be refactored so that things like collapsible headings can be implemented on top of core. In which case, some development could still happen on this repo or maybe that could now all happen in core. Others invariably know more than I.

@telamonian
Copy link
Member Author

some development could still happen on this repo

That kind of splitting of issues and codebase is exactly what I want to avoid. The TOC extension included in core is now the canonical one, that's where all new work should go

or maybe that could now all happen in core

yes, please

@marthacryan
Copy link
Member

I support archiving this - as mentioned above, it's better to just direct all changes to the core extension now. Not sure I have the permissions to archive it, but I'll add a PR to add the text to the README about versions / etc.

@ellisonbg
Copy link

ellisonbg commented Nov 20, 2020 via email

@jasongrout
Copy link
Contributor

This is saying that there will be no more releases of ToC for jlab 2.x, even backports of things from the core 3.0 extension. Is that okay with everyone? Especially given that we still haven't released 3.0 and 2.x is still the current version.

@jasongrout
Copy link
Contributor

It makes me a bit nervous to archive something that is still the stable released version, without the next version being released yet

@lresende
Copy link
Member

How about we clarify the direction we are taking in the README, and wait a little bit for the archive.
Having said that, I don't have access to archive it.

@jasongrout
Copy link
Contributor

How about we clarify the direction we are taking in the README, and wait a little bit for the archive.

That seems safer to me. If there is a huge security issue that needs a patch in the jlab 2.x plugin, you might still want to do a patch release. I would say archive the repo when the 2.x plugin is no longer supported.

@marthacryan
Copy link
Member

marthacryan commented Nov 20, 2020

@jasongrout Do you think we should be backporting PR's from the core 3.0 extension? I haven't been doing that for any of my PR's in the main JL repo, but I can do that if you want. Good point that security releases could be necessary though so we should wait until 2.x is no longer supported to archive. I can update the README to explain it correctly

@jasongrout
Copy link
Contributor

jasongrout commented Nov 20, 2020

Do you think we should be backporting PR's from the core 3.0 extension?

That's up to the maintainers here to make that decision, of course (i.e., including you :), but my feeling is that just backporting security fixes is perfectly fine.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

6 participants