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

Update project status in README #6117

Merged
merged 1 commit into from May 23, 2018

Conversation

sdogruyol
Copy link
Member

The Status section in the README hasn't been updated since ages. Crystal has come a long way and we should not call it alpha anymore.

@Sija
Copy link
Contributor

Sija commented May 22, 2018

I'd suggest adding [ci skip] prefix to the commit message.

README.md Outdated
------

* The project is in alpha stage: we are still tweaking the language and standard library.
* The compiler is written in Crystal.
Crystal is still under heavy development. There can be breaking changes but we're trying to keep those as minimum as possible.
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you add a - before this and add this under it: "The compiler uses LLVM 4.0 to compile to native code."? Because I think it could be interesting to know which LLVM version Crystal is currently using.
And also maybe "Windows is currently not supported. It's a work in progress."

Copy link
Contributor

Choose a reason for hiding this comment

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

"The compiler uses LLVM 4.0 to compile to native code."? Because I think it could be interesting to know which LLVM version Crystal is currently using.

I'd skip it, since it's a moving target, mine is built with LLVM 5.0.1 for instance.

Copy link
Contributor

Choose a reason for hiding this comment

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

"those as minimum as possible" should be "them as minimal as possible"

Copy link
Contributor

Choose a reason for hiding this comment

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

/cc @RX14

Copy link
Contributor

@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.

@sdogruyol sdogruyol merged commit 0cc4c34 into crystal-lang:master May 23, 2018
@RX14 RX14 added this to the Next milestone May 23, 2018
chris-huxtable pushed a commit to chris-huxtable/crystal that referenced this pull request Jun 6, 2018
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.

None yet

9 participants