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

Enable expeditor #130

Merged
merged 3 commits into from
Jul 31, 2018

Conversation

dheerajd-msys
Copy link
Contributor

Signed-off-by: dheerajd-msys [email protected]

Signed-off-by: dheerajd-msys <[email protected]>
@dheerajd-msys dheerajd-msys force-pushed the dh/MSYS-852_expeditor_support branch from a1fb463 to edf725d Compare July 10, 2018 09:53
.travis.yml Outdated
- 2.4.1
- 2.3.4
- 2.4.2
Copy link

@Vasu1105 Vasu1105 Jul 25, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add versions in order.

Signed-off-by: dheerajd-msys <[email protected]>
@dheerajd-msys dheerajd-msys force-pushed the dh/MSYS-852_expeditor_support branch from edf725d to 134303d Compare July 25, 2018 07:22
@dheerajd-msys dheerajd-msys changed the title [MSYS-852]enable expeditor Enable expeditor Jul 25, 2018
@tas50
Copy link
Contributor

tas50 commented Jul 30, 2018

@Vasu1105 Any chance you can update this for the latest release?

Signed-off-by: dheerajd-msys <[email protected]>
@dheerajd-msys
Copy link
Contributor Author

Hi @tas50, now this is updated for latest release 3.3.0

@@ -3,9 +3,9 @@ cache: bundler
dist: trusty
sudo: false
rvm:
- 2.2.7
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This got a big mangled here

@tas50 tas50 merged commit e5ae934 into chef:master Jul 31, 2018
@btm
Copy link
Contributor

btm commented Aug 6, 2018

expeditor didn't fire here #134. I looked and saw that 'Service Accounts' was missing from collaborators and added it.

I made an empty commit here: #135 but that didn't fire. There was an upgrade of expeditor around that time, maybe that's why.

Another empty commit here worked: #136

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.

4 participants