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.

Sign up for GitHub

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

Jump to bottom

Fixes reference stack recommendation to pick the best reference stack #3

Merged
merged 1 commit into from
May 5, 2017
Merged

Fixes reference stack recommendation to pick the best reference stack #3

merged 1 commit into from
May 5, 2017

Conversation

miteshvp
Copy link
Contributor

@miteshvp miteshvp commented May 2, 2017

No description provided.

@bkabrda
Copy link
Contributor

bkabrda commented May 5, 2017

Please rebase, we've been doing changes to not point at the old baytemp org, but you sent the PR too soon and Jenkins still sees the old links and fails because of that. Rebasing on top of latest master should fix this.

@bkabrda
Copy link
Contributor

bkabrda commented May 5, 2017

Otherwise this LGTM, so once Jenkins run passes, I can merge.

@miteshvp miteshvp force-pushed the fix_recommendation branch from 36b9d84 to e3a0289 Compare May 5, 2017 06:47
@bkabrda
Copy link
Contributor

bkabrda commented May 5, 2017

Hmm, ok, I see that the problem is elsewhere and I opened #8 to fix it. This looks fine, so I'm merging.

@bkabrda bkabrda merged commit 5ec3c79 into fabric8-analytics:master May 5, 2017
yzainee-zz pushed a commit to yzainee-zz/fabric8-analytics-worker that referenced this pull request Nov 29, 2018
@msehnout
# This is a combination of 17 commits.
a4b3e9b 
# This is the 1st commit message:

Merge pull request fabric8-analytics#742 from msehnout/remove-keywords-workers

remove keywords tagger and summary tasks
# This is the commit message fabric8-analytics#2:

initial commit for git operations addition in worker

# This is the commit message fabric8-analytics#3:

adding logs

# This is the commit message fabric8-analytics#4:

lint

# This is the commit message fabric8-analytics#5:

file rectified

# This is the commit message fabric8-analytics#6:

rectified queue name

# This is the commit message fabric8-analytics#7:

added functionanilty for git operations

# This is the commit message fabric8-analytics#8:

lint

# This is the commit message fabric8-analytics#9:

rectified clone

# This is the commit message fabric8-analytics#10:

additional logic for clone

# This is the commit message fabric8-analytics#11:

token rectified

# This is the commit message fabric8-analytics#12:

gh username rectified

# This is the commit message fabric8-analytics#13:

rectified gemini call

# This is the commit message fabric8-analytics#14:

added logs

# This is the commit message fabric8-analytics#15:

rectified parser logic

# This is the commit message fabric8-analytics#16:

modified logic for gemini

# This is the commit message fabric8-analytics#17:

host resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@miteshvp @bkabrda