-
Notifications
You must be signed in to change notification settings - Fork 104
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
chore: split structure #558
Conversation
497da1e
to
d36dbc7
Compare
bd8ec08
to
a92e79a
Compare
05e58f4
to
963ae88
Compare
I'll be merging, please @RafaelGSS if you have any feedback I'll address it in a further PR 👍 |
push (task : Task) : void { | ||
this.tasks.push(task); | ||
} | ||
const { version } = JSON.parse( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@metcoder95 hi, this is currently breaking things for us.
Is this version
extraction required? can we pass in a custom path or version in options instead?
This breaks flexibility in a number of ways, like using esbuild and bundling.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let me revert, this is causing too many problems. Will open a PR and publish a fix soon
Attempt to try to reduce the complexity of the project with the goal of ease of understanding