Skip to content

Commit

Permalink
Add #609 to changelogs
Browse files Browse the repository at this point in the history
  • Loading branch information
KronicDeth committed Feb 10, 2017
1 parent 3226670 commit f388821
Show file tree
Hide file tree
Showing 2 changed files with 43 additions and 0 deletions.
12 changes: 12 additions & 0 deletions CHANGELOG.md
Expand Up @@ -126,6 +126,12 @@
* Show `OtpErlangBitStr` (and therefore `OtpErlangBinary` contents when tests fail
* Quote binaries as `to_charlist` instead of `to_char_list` for Elixir `>= 1.3`. Depends on Elixir version of project SDK.
* Use `elixir` instead of `java` VM, so now Erlang and Elixir don't need to be built on travis-ci, but `ant` and the `jdk` need to be installed, but unlike Erlang and Elixir, there are tarballs for that, so this way is faster than the old method without depending on travis-ci cache.
* [#609](https://github.com/KronicDeth/intellij-elixir/pull/609) - [@KronicDeth](https://github.com/KronicDeth)
* If `multiResolve` causes a `StackOverflow` for `org.elixir_lang.annotator.Callable.visitCall`, then `catch` it and use `errorreport` logger to log the element.
* Include file path in `errorreport` excerpt
* Log element for `StackOverflow` related to `import`s
* Regression test for [#605](https://github.com/KronicDeth/intellij-elixir/issues/605).
* Log `LookupElement#getObject` when `LookupElement#getPsiElement` is `null` to track down how it was `null` in [#563](https://github.com/KronicDeth/intellij-elixir/issues/563).

### Bug Fixes
* [#574](https://github.com/KronicDeth/intellij-elixir/pull/574) - Fix copy-paste errors in `MatchOperatorInsteadOfTypeOperator` - [@KronicDeth](https://github.com/KronicDeth)
Expand All @@ -146,6 +152,12 @@
* [#600](https://github.com/KronicDeth/intellij-elixir/pull/600) - [@KronicDeth](https://github.com/KronicDeth)
* Check children of `MultipleAliases` for variable declarations.
* Treat any variable declared in a `MultipleAliases` as invalid.
* [#609](https://github.com/KronicDeth/intellij-elixir/pull/609) - [@KronicDeth](https://github.com/KronicDeth)
* Skip `import Kernel` in `kernel.ex` to prevent stack overflow due to recursive `import`
* Strip all outer parentheses for left type operand, so that `(+value)` can be see as `+` operator type spec.
* Use advice from `IndexNotReadyException` documentation and check `DumbService.isDumb(Project)` before calling `StubIndex.getElements` in `Module` and `module.MultiResolve.indexNameElements`.
* Don't `assert` that `LookupElement#getPsiElement` is not `null` in `CallDefinitionCluase.renderElement`
* Update to `ant` `1.10.1` because `1.10.0` is no longer hosted.

### Incompatible Changes
* [#585](https://github.com/KronicDeth/intellij-elixir/pull/585) - Move `^^^` to its own three-operator precedence level to match `1.2`. This does mean the parsing will be wrong for Elixir `1.1`, but is simpler than maintaining two grammars for those that are still using Elixir `1.1` - [@KronicDeth](https://github.com/KronicDeth)
Expand Down
31 changes: 31 additions & 0 deletions src/META-INF/plugin.xml
Expand Up @@ -66,6 +66,19 @@
Elixir, there are tarballs for that, so this way is faster than the old method without depending on
Travis-CI cache.
</li>
<li>
If <code>multiResolve</code> causes a <code>StackOverflow</code> for
<code>org.elixir_lang.annotator.Callable.visitCall</code>, then <code>catch</code> it and use
<code>errorreport</code> logger to log the element.
</li>
<li>Include file path in <code>errorreport</code> excerpt</li>
<li>Log element for <code>StackOverflow</code> related to <code>import</code>s</li>
<li>Regression test for <a href="https://github.com/KronicDeth/intellij-elixir/issues/605">#605</a>.</li>
<li>
Log <code>LookupElement#getObject</code> when <code>LookupElement#getPsiElement</code> is
<code>null</code> to track down how it was <code>null</code> in
<a href="https://github.com/KronicDeth/intellij-elixir/issues/563">#563</a>.
</li>
</ul>
</li>
<li>
Expand Down Expand Up @@ -109,6 +122,24 @@
</li>
<li>Check children of <code>MultipleAliases</code> for variable declarations.</li>
<li>Treat any variable declared in a <code>MultipleAliases</code> as invalid.</li>
<li>
Skip <code>import Kernel</code> in <code>kernel.ex</code> to prevent stack overflow due to recursive
<code>import</code>
</li>
<li>
Strip all outer parentheses for left type operand, so that <code>(+value)</code> can be see as
<code>+</code> operator type spec.
</li>
<li>
Use advice from <code>IndexNotReadyException</code> documentation and check
<code>DumbService.isDumb(Project)</code> before calling <code>StubIndex.getElements</code> in
<code>Module</code> and <code>module.MultiResolve.indexNameElements</code>.
</li>
<li>
Don't <code>assert</code> that <code>LookupElement#getPsiElement</code> is not <code>null</code> in
<code>CallDefinitionCluase.renderElement</code>
</li>
<li>Update to <code>ant</code> <code>1.10.1</code> because <code>1.10.0</code> is no longer hosted.</li>
</ul>
</li>
<li>
Expand Down

0 comments on commit f388821

Please sign in to comment.