Skip to content

docs: suggest import mapping for newer versions#114

Merged
Kehrlann merged 2 commits intoilearnio:devfrom
Nytelife26:dev
Nov 3, 2025
Merged

docs: suggest import mapping for newer versions#114
Kehrlann merged 2 commits intoilearnio:devfrom
Nytelife26:dev

Conversation

@Nytelife26
Copy link
Copy Markdown

For more context, see #113

Copy link
Copy Markdown
Collaborator

@Kehrlann Kehrlann left a comment

Choose a reason for hiding this comment

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

Looks good to me, although to be fair I haven't tried it myself. Do you have a sample project so I can take a look?

Tagging @ilearnio to review.

Comment thread README.md Outdated
Comment thread README.md
@Kehrlann Kehrlann self-requested a review February 22, 2021 08:26
Copy link
Copy Markdown
Collaborator

@Kehrlann Kehrlann left a comment

Choose a reason for hiding this comment

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

See small formatting suggestions.

@Kehrlann Kehrlann requested a review from ilearnio February 22, 2021 08:30
@Nytelife26 Nytelife26 requested a review from Kehrlann February 22, 2021 14:27
@Nytelife26
Copy link
Copy Markdown
Author

@Kehrlann Can you dismiss your stale review? Unfortunately, for some reason, I cannot.

@Nytelife26
Copy link
Copy Markdown
Author

Status on this?

@Kehrlann Kehrlann merged commit d92f3de into ilearnio:dev Nov 3, 2025
@Kehrlann
Copy link
Copy Markdown
Collaborator

Kehrlann commented Nov 3, 2025

Sorry, this project completely fell out of sight 😓

@Nytelife26
Copy link
Copy Markdown
Author

That's okay, welcome back @Kehrlann. I hope you're doing well. 5 years down the line, I believe the best course of action would be to add a deprecation notice and archive the repository. The versions of Node that originally added path aliasing and subpath imports have been end-of-life for multiple years now.

@Kehrlann
Copy link
Copy Markdown
Collaborator

Kehrlann commented Nov 3, 2025

I agree, but I don’t have control over this repo.

@ilearnio what do you think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants