Skip to content
This repository has been archived by the owner on Jun 1, 2023. It is now read-only.

Fixes autocompletion when constructor has integer or boolean params #77

Merged
merged 3 commits into from Apr 10, 2018

Conversation

laginha87
Copy link
Contributor

Fixes the following issue.
On this scenario:

a = A.new(1)
a.

The current implementation identifies the type of a as an integer when it should be A.

@faustinoaq faustinoaq added this to To do in Crystal Tools via automation Apr 9, 2018
@faustinoaq faustinoaq requested a review from a team April 9, 2018 08:08
@faustinoaq faustinoaq moved this from To do to In progress in Crystal Tools Apr 9, 2018
@faustinoaq
Copy link
Member

@laginha87 Looks like Travis found some formatter changes, see: https://travis-ci.org/crystal-lang-tools/scry/jobs/363996490#L520

@faustinoaq
Copy link
Member

@crystal-lang-tools/scry I guess this is the last PR before releasing a new version 😅

Should we release v0.7.2 or v0.8.0 ?

@keplersj
Copy link
Contributor

keplersj commented Apr 9, 2018

v0.8.0

Copy link
Member

@faustinoaq faustinoaq left a comment

Choose a reason for hiding this comment

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

Looks nice 👍

I approve if Travis is ok

@laginha87 laginha87 changed the title Adds a failing spec Fixes autcompletion and constructor has integer or boolean params Apr 9, 2018
@laginha87 laginha87 changed the title Fixes autcompletion and constructor has integer or boolean params Fixes autocompletion and constructor has integer or boolean params Apr 9, 2018
@laginha87 laginha87 changed the title Fixes autocompletion and constructor has integer or boolean params Fixes autocompletion when constructor has integer or boolean params Apr 10, 2018
@bararchy
Copy link

@faustinoaq seems travis is ok and so are you, LGTM ?

@faustinoaq
Copy link
Member

@bararchy Yeah, I approved, just waiting another approval to merge it 👍

@faustinoaq faustinoaq merged commit bf2660b into master Apr 10, 2018
Crystal Tools automation moved this from In progress to Done Apr 10, 2018
@faustinoaq faustinoaq deleted the laginha87-fix-integer-completion branch April 12, 2018 01:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Crystal Tools
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants