-
-
Notifications
You must be signed in to change notification settings - Fork 135
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
TASK: Exchange yarn with make as build tool #1633
Conversation
Hey Max, thanks! I'm not opinionated about tooling at all, I'm fine with whatever works, but... why make? I've never seen it used in JS projects... What advatages does it bring over plain npm scripts? |
699e4d3
to
0ee407c
Compare
Rebased to remove |
Is this still WIP? |
This is still WIP because I didn't update the docs according to this change. I want at least one approval to this change from someone who has an opinion to not decide alone on our build stack. :) |
@dimaip @mstruebing i simply love it! |
@mstruebing so you get a +1 from @johannessteu and +0.(6) from me, merge it? |
@dimaip first adjust the documentation ;) I'm on it. |
I'm done! :) |
open a pull request on Github. When travis runs through merge it an tag a new | ||
release on Github. | ||
|
||
After that trigger jenkins with the new version. |
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.
Could Jenkins do everything, including make release
?
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.
In general yeah. We need to give him access to master branch and should find some way to only commit stuff we expect to change
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.
Master is generally protected, I don't know how to change that especially only for jenkins, we should not be able to push into master.
references #1629
I would update the docs if we decide to merge it.
I would love to hear what you people think of this.