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

TASK: Exchange yarn with make as build tool #1633

Merged
merged 12 commits into from
Feb 13, 2018
Merged

Conversation

mstruebing
Copy link
Contributor

references #1629

I would update the docs if we decide to merge it.
I would love to hear what you people think of this.

@mstruebing mstruebing changed the title TASK: Exchange yarn with make as build tool WIP TASK: Exchange yarn with make as build tool Feb 11, 2018
@dimaip
Copy link
Contributor

dimaip commented Feb 11, 2018

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?
In any case I think it might make sense to wrap its functionality via npm scripts anyways, no?

@mstruebing mstruebing force-pushed the make branch 2 times, most recently from 699e4d3 to 0ee407c Compare February 13, 2018 05:27
@mstruebing
Copy link
Contributor Author

Rebased to remove .nvmrc and working fine :)

@dimaip
Copy link
Contributor

dimaip commented Feb 13, 2018

Is this still WIP?
@johannessteu what do you think about this PR?

@mstruebing
Copy link
Contributor Author

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. :)

@johannessteu
Copy link
Contributor

@dimaip @mstruebing i simply love it!

@dimaip
Copy link
Contributor

dimaip commented Feb 13, 2018

@mstruebing so you get a +1 from @johannessteu and +0.(6) from me, merge it?

@mstruebing
Copy link
Contributor Author

@dimaip first adjust the documentation ;)

I'm on it.

@mstruebing mstruebing changed the title WIP TASK: Exchange yarn with make as build tool TASK: Exchange yarn with make as build tool Feb 13, 2018
@mstruebing
Copy link
Contributor Author

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.
Copy link
Contributor

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?

Copy link
Contributor

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

Copy link
Contributor Author

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.

@mstruebing mstruebing merged commit a6e5057 into neos:master Feb 13, 2018
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