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

Remove false advertising #215

Merged
merged 1 commit into from
Jun 19, 2024
Merged

Remove false advertising #215

merged 1 commit into from
Jun 19, 2024

Conversation

AnthonyFuller
Copy link
Contributor

As title.

@quantix-dev
Copy link

I think this also needs changing on the documentation because I got misled into thinking it was ~3MB
image

@filiptibell
Copy link
Collaborator

The README is outdated and Lune has grown substantially since it was last updated - no false advertising was intentional on my end, I'm sorry if it came across that way. We should probably just remove mentions of a specific binary size altogether since I don't expect Lune to stop growing at this moment in time, if you do want a smaller binary you can compile from source with --no-default-features and enable only the libraries / built-ins you want

@AnthonyFuller
Copy link
Contributor Author

AnthonyFuller commented Jun 17, 2024

Hello! I have just posted some messages in your Discord! But I'll add it here:

I'd like to make clear that it was a joke. I'm not actually accusing you of false advertising. I definitely could've been more clear on it, but at the time I did not expect it to be perceived as an attack.

Everyone was upvoting it because they found it funny, not really because they agree.

Verde (I have no idea what Azul is) is not even related to this.

Which might I add is an alternative system, not a hard fork [of Rojo].

EDIT: Feel free to squash and merge it under another name, for example: chore: update file size in read me.

@filiptibell filiptibell merged commit 430d568 into lune-org:main Jun 19, 2024
5 of 6 checks passed
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.

3 participants