Skip to content

Webpack Alias: Added Lib alias#303

Closed
shmolf wants to merge 1 commit intoBoostIO:masterfrom
shmolf:lib-alias
Closed

Webpack Alias: Added Lib alias#303
shmolf wants to merge 1 commit intoBoostIO:masterfrom
shmolf:lib-alias

Conversation

@shmolf
Copy link
Copy Markdown

@shmolf shmolf commented Jan 18, 2020

I've used webpack alias before, and it helps to make importing cleaner, and more direct.
Here's the reference I've used before.

I've loaded the application and created a new folder, and a couple of notes. They store and are retrieved as expected.

There were no errors generated, apart from the missing-environment-variable warnings.

The benefit is, instead of referencing ../../lib/ or ../../../../../lib/, developers can simply reference Lib/.


The significant changes are in

  • tsconfig.json
  • webpack.config.ts

@claassistantio
Copy link
Copy Markdown

claassistantio commented Jan 18, 2020

CLA assistant check
All committers have signed the CLA.

@Rokt33r
Copy link
Copy Markdown
Member

Rokt33r commented Jan 18, 2020

@shmolf Thanks for submitting this PR. I like the idea. But, sadly, I should close this issue for now because I'm working on huge refactoring. But we'll be back to this issue one day after the app's code base becomes stable enough.

@Rokt33r Rokt33r closed this Jan 18, 2020
@shmolf
Copy link
Copy Markdown
Author

shmolf commented Jan 18, 2020

Sounds appropriate. Once it's stable enough, I can pull a fresh master copy, and re-implement.

Thanks for the update.

@shmolf
Copy link
Copy Markdown
Author

shmolf commented Feb 15, 2020

@Rokt33r I know you were doing some refactoring. Mind if I create some alias, without changing any files to use them? In this way, new files can reference the alias, and later, when the codebase slows down, I can refactor what remains

@Rokt33r
Copy link
Copy Markdown
Member

Rokt33r commented Feb 21, 2020

@shmolf I still want to wait until I really need this. I still can do refactor to make module structure simpler.

But I'll definitely introduce the module alias when I feel I can do improve any better without it.

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.

3 participants