P. L. Lim comments

Results 2472 comments of


                                            P. L. Lim

build(deps): typing-extensions as a dependency

Still, doc build and mypyc aren't something end-user need to worry about, so why not optional dev dependency, why required?

build(deps): typing-extensions as a dependency

What about here? https://github.com/astropy/astropy/blob/ec2d974fbf1880bb8b096ee3d818abda62764c36/pyproject.toml#L123

build(deps): typing-extensions as a dependency

"no big deal" is not a precedent we should set. Please discuss with community. We have time as this is still very early in the release cycle. Thanks! * astropy-dev...

Add version changed for API changes

Since this is marked as "novice" and "good first issue", would be nice to give some ideas on how to `grep` for the doc texts that need this extra directive.

Ingest the STScI notebooks to the index

The repo mentioned is no longer active. `Content in this repository is not maintained and is actively being migrated to other repos.`

Ingest the STScI notebooks to the index

So what is the intention? There are many repos now. Is your interest just JWST or... ? Please clarify. Thanks.

Ingest the STScI notebooks to the index

@camipacifici , any advise on which notebook repos fits the description of https://github.com/astropy/learn-astropy-librarian/issues/9#issuecomment-2411946867 ? Thanks!

API: change default behavior in `Table.pformat` and `Column.pformat` and add (pending) deprecation to `Table.pformat_all`

@taldcroft , you need to approve for auto merge to happen. 😉

API: change default behavior in `Table.pformat` and `Column.pformat` and add (pending) deprecation to `Table.pformat_all`

Well, if you keep the auto-merge, it would have merged as soon as approval is granted.

MNT: Run PTH flake test in prep for supporting pathlib (wcs)

Given the recent revert of PTH changes, let's not do this for now. Thanks anyway!