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

[barrel] Use modern namespace re-export #5

Open
jasonkuhrt opened this issue Apr 4, 2022 · 3 comments
Open

[barrel] Use modern namespace re-export #5

jasonkuhrt opened this issue Apr 4, 2022 · 3 comments

Comments

@jasonkuhrt
Copy link

// codegen:start {preset: barrel, include: ./sources/index.ts, import: star }
import * as sources from './sources/index'

export { sources }
// codegen:end

Could be:

// codegen:start {preset: barrel, include: ./sources/index.ts, import: star }
export * as sources from './sources/index'
// codegen:end
@mmkal mmkal changed the title Use modern namespace re-export [eslint-plugin-codegen] Use modern namespace re-export Sep 25, 2022
@mmkal
Copy link
Owner

mmkal commented Oct 21, 2022

Makes sense - note that you can write a custom script which does this right now. I'm moving this to a new repo soon, I'll port this issue over when that happens.

@mmkal mmkal transferred this issue from mmkal/ts Jan 2, 2023
@mmkal
Copy link
Owner

mmkal commented Apr 6, 2023

@jasonkuhrt the repo is now moved - would you like to create a PR for this?

@jasonkuhrt
Copy link
Author

Hey, don't think I need this enough for the PR. But good to know you're open to it if ever!

@mmkal mmkal changed the title [eslint-plugin-codegen] Use modern namespace re-export Use modern namespace re-export Nov 6, 2023
@mmkal mmkal changed the title Use modern namespace re-export [barrel] Use modern namespace re-export Nov 6, 2023
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

No branches or pull requests

2 participants