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

Removed sorting of items for auto completion #114

Merged
merged 2 commits into from
Dec 6, 2018
Merged

Removed sorting of items for auto completion #114

merged 2 commits into from
Dec 6, 2018

Conversation

SamuelBrucksch
Copy link
Contributor

@SamuelBrucksch SamuelBrucksch commented Nov 27, 2018

vscode deals with sorting the items in the auto completion list so no need to do it here. Might improve speed a bit if this does not have to be done on each request.

Not sure if i miss something. If i do please illuminate me.

@SamuelBrucksch
Copy link
Contributor Author

related to #95

I have some more ideas to improve auto completion speed, however i need to investigate if they work out.

@Confectrician
Copy link
Collaborator

Hi @SamuelBrucksch,

I am not a pro in debugging PR code, so please give me some time.
(Kuba did the most work here and i am finding myself into the typescipt/extension stuff alone currently.)

One thing i can already tell you, is that you need to add a sign off statement to your commit.
(If you need help with that git stuff, write me an answer here.)

See our Contribution Guidelines for more informatoin about signing of and what is needed to merge something into openHAB repositories.

BR
Jerome

@Confectrician
Copy link
Collaborator

Hey @SamuelBrucksch,

I had a look in the debugger and would like to merge this one.
Do you need any additional information for dooing the sign off statement?

BR
Jerome

@SamuelBrucksch
Copy link
Contributor Author

Hey, will look into it tomorrow or the day after. Cheers Sam

@SamuelBrucksch
Copy link
Contributor Author

SamuelBrucksch commented Dec 6, 2018

Hi,

i added the signed by and also tried it with git commit --signoff but the check still fails. Am i doing something wrong here? Is it maybe because of the push -f and i should add another commit?

BR
Samuel

SamuelBrucksch and others added 2 commits December 6, 2018 09:20
vscode deals with sorting the items in the auto completion list so no need to do it here. might improve speed a bit.

Signed-off-by: Samuel Brucksch <sasliga@freenet.de>

Signed-off-by: Samuel Brucksch <SamuelBrucksch@users.noreply.github.com> (github: SamuelBrucksch)
Signed-off-by: Samuel Brucksch <sasliga@freenet.de> (github: SamuelBrucksch)
@Confectrician
Copy link
Collaborator

Confectrician commented Dec 6, 2018

Hm,

No Idea why it still stucks. Maybe the force did confuse it.
But pullaprove is just a check.
Thankfully the mighty git gods gave me some accesses to overrule that.

The sign-off looks good to me, so LGTM.

@SamuelBrucksch
Copy link
Contributor Author

Thanks, will check again later if it works on a new PR. Maybe there was something wrong in here because i ammended the sign off.

@Confectrician
Copy link
Collaborator

Ammend should be the correct way.

But you have 2 sign off statements in the first commit.
The last one is the one with the "users.github" email. Maybe this blocks the pullapprove mechanism.
I don't know if pullapprove allows github noreply email adresses.

If you do the signoff directly you shouldn't have problems.
Works for me since ages.

Just check if your get config is up to date git config user.name & git config user.email and git commit -s -m "MYCOMMITMESSAGE" should work fine and append the sign off part.

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.

2 participants