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

Multi root adoption #347

Closed
wants to merge 3 commits into from
Closed

Multi root adoption #347

wants to merge 3 commits into from

Conversation

aeschli
Copy link
Collaborator

@aeschli aeschli commented Oct 30, 2017

No description provided.

@fbricon
Copy link
Collaborator

fbricon commented Oct 31, 2017

@aeschli can you sign-off your last commit please?

@@ -58,15 +58,17 @@
"null"
Copy link
Collaborator

Choose a reason for hiding this comment

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

should we add the multi-root ready keyword as recommended or is it too early?

@aeschli
Copy link
Collaborator Author

aeschli commented Nov 1, 2017

Not sure why this doesn't work.
image

Signed-off-by: Martin Aeschlimann <martinae@microsoft.com>
@aeschli
Copy link
Collaborator Author

aeschli commented Nov 1, 2017

ok, all commits need to be signed (in eclipse.jdt.ls it just has to be the last).
That check is really a hassle. Please remove it again. I thought by becoming a contributor I've already agreed that all I commit is originated by me.

@fbricon
Copy link
Collaborator

fbricon commented Nov 1, 2017

@gorkem ^^

@fbricon
Copy link
Collaborator

fbricon commented Nov 6, 2017

rebased as 5a0ef76

@fbricon fbricon closed this Nov 6, 2017
@fbricon fbricon added this to the Mid November 2017 milestone Nov 6, 2017
@aeschli aeschli deleted the multi-root-adoption branch November 7, 2017 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants