Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [che-dev] ci-build-check now required

@Florent Benoit we didn't request require of branch being up-to-date seems eclipse org enabled this automatically. I was not aware that it was enabled for CHE repos but I think this is very important and thing to have.

@Ilya Buziuk I will check if you are in the white list

On Mon, Nov 26, 2018 at 11:25 AM Ilya Buziuk <ibuziuk@xxxxxxxxxx> wrote:
I also noticed it while reviewing https://github.com/eclipse/che/pull/11989
pressing update branch creates a merge commit, which breaks ip validation 
also, it seems that  ci-build does not trigger the build check for me

On Mon, Nov 26, 2018 at 10:01 AM Florent Benoit <florent@xxxxxxxxxx> wrote:
Roman, 
I noticed that the buttons to merge are also requiring to merge with latest. (AFAIK It was not discussed previously)

an example : To merge a typo PR from some an external contributor, this mandatory 'merge' is not working at all as it's requiring contributors to update their pull request at each new commit to master branch.

Regards,

Florent

On Wed, Nov 21, 2018 at 10:18 AM Roman Iuvshyn <riuvshyn@xxxxxxxxxx> wrote:
hello, 
please be informed that now ci-build-check is required for every PR against eclipse/che repository this check will be started automatically and can be restarted by ci-build trigger phrase in the PR comment.

ci-test is still required to run for most PRs


--

ROMAN IUVSHYN

Red Hat 

riuvshyn@xxxxxxxxxx   

_______________________________________________
che-dev mailing list
che-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://www.eclipse.org/mailman/listinfo/che-dev
_______________________________________________
che-dev mailing list
che-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://www.eclipse.org/mailman/listinfo/che-dev


--

ILYA BUZIUK

SENIOR SOFTWARE ENGINEER

Red Hat 

ibuziuk@xxxxxxxxxx   

_______________________________________________
che-dev mailing list
che-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://www.eclipse.org/mailman/listinfo/che-dev


--

ROMAN IUVSHYN

Red Hat 

riuvshyn@xxxxxxxxxx   


Back to the top