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

Corrected typo, set Compiler to 90% #314

Merged
merged 2 commits into from
Apr 11, 2018
Merged

Conversation

ShalokShalom
Copy link
Contributor

@ShalokShalom ShalokShalom commented Mar 28, 2018

I added a percentage sign to the Elchemy-core support.

So this also changes the percentage support for the compiler to 90% since no Windows support weights probably more as 1% in practically everybody's opinion.

Wonderful to see how this overview about your progress grows, I absolutely love it ^-^

I added a percentage sign and I think 'no Windows support' weights more as 1%
Spelling that a compiler is 99% ready with zero support for this one OS might seem a little bit like discrimination? 
Wonderful to see how this overview about your progress grows, I absolutely love it ^-^
README.md Outdated
@@ -161,8 +161,8 @@ Elchemy uses Elm to typecheck your program. Although it is possible to use it wi
# Maturity of the project

- Parser - **99%** of Elm's syntax (see [elm-ast](https://github.com/Bogdanp/elm-ast/issues))
- Compiler - **99%** (Sophisticated incremental compilation. No support for Windows yet though ([#287](https://github.com/wende/elchemy/issues/287)) also big reliance on unix tools ([#288](https://github.com/wende/elchemy/issues/288))
- Elchemy-core - **95** ( Everything covered except side effects and JSON Decoders)
- Compiler - **90%** (Sophisticated incremental compilation. No support for Windows yet though ([#287](https://github.com/wende/elchemy/issues287)) also big reliance on unix tools ([#288](https://github.com/wende/elchemy/issues/288))
Copy link
Owner

Choose a reason for hiding this comment

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

The second change breaks the link. Plus it is obviously arguable, but I don't consider out of the box windows support as a compiler feature

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Well, you mention it a few characters next to the percentage, so it suggests that you see it so?
I fixed the link, good catch ;)

@wende
Copy link
Owner

wende commented Apr 9, 2018

LGTM 👍

@ShalokShalom
Copy link
Contributor Author

Is this LGTM supposed to trigger the merge?
Since its still open.

@wende wende merged commit d5da433 into wende:master Apr 11, 2018
wende added a commit that referenced this pull request May 16, 2018
================

  * Closes #276 macro plugins (#331)
  * Update elm-package.json
  * Bug: Closes #327 (#328)
  * Closes #329 - Warn when elchemy not installed during project compilation (#330)
  * Closes #238 - Debug.log print in Elchemy format instead of Elixir format (#325)
  * Closes #322 Warn when elchemy line put incorrectly (#323)
  * Update README.md (#320)
  * correct typos (#318)
  * type savety and Elixir (#310)
  * Closes #313 Fixes interop in gitbook and new gitbbok (#317)
  * Corrected typo, set Compiler to 90% (#314)
  * Corrected typo (#306)
  * Corrected typos (#307)
  * correct typo (#308)
  * Delete FEATURES.MD
  * Create CODE_OF_CONDUCT.md (#304)
  * Typo in gitbook (#305)
  * 0.6.6 / 2018-03-05 ================
  * 0.6.6
  * format properly
  * 0.6.5 / 2018-03-05 ================
  * 0.6.5
  * format when > 1.6.0 (#300)
  * Update README.md
  * Better references in maturity of the project summary
  * Update SIDE_EFFECTS.md
  * Update SUMMARY.md
  * Update SUMMARY.md
  * Add SYNTAX OVERVIEW
  * Create SYNTAX.md
  * Fix to 292 (#293)
  * Closes #290 - incremental compiler bug  (#291)
  * 0.6.4 / 2018-02-07 ================
  * 0.6.4
  * Updated core
wende added a commit that referenced this pull request May 23, 2018
================

  * Clean makes init not able to work (#341)
  * No project name (#339)
  * Correct typos in roadmap/STRUCTURES.md (#319)
  * Added formating and test (#337)
  * 0.7.2 / 2018-05-16 ================
  * 0.7.2
  * Closes #276 macro plugins (#331)
  * Update elm-package.json
  * Bug: Closes #327 (#328)
  * Closes #329 - Warn when elchemy not installed during project compilation (#330)
  * Closes #238 - Debug.log print in Elchemy format instead of Elixir format (#325)
  * Closes #322 Warn when elchemy line put incorrectly (#323)
  * Update README.md (#320)
  * correct typos (#318)
  * type savety and Elixir (#310)
  * Closes #313 Fixes interop in gitbook and new gitbbok (#317)
  * Corrected typo, set Compiler to 90% (#314)
  * Corrected typo (#306)
  * Corrected typos (#307)
  * correct typo (#308)
  * Delete FEATURES.MD
  * Create CODE_OF_CONDUCT.md (#304)
  * Typo in gitbook (#305)
  * 0.6.6 / 2018-03-05 ================
  * 0.6.6
  * format properly
  * 0.6.5 / 2018-03-05 ================
  * 0.6.5
  * format when > 1.6.0 (#300)
  * Update README.md
  * Better references in maturity of the project summary
  * Update SIDE_EFFECTS.md
  * Update SUMMARY.md
  * Update SUMMARY.md
  * Add SYNTAX OVERVIEW
  * Create SYNTAX.md
  * Fix to 292 (#293)
  * Closes #290 - incremental compiler bug  (#291)
  * 0.6.4 / 2018-02-07 ================
  * 0.6.4
  * Updated core
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