-
Notifications
You must be signed in to change notification settings - Fork 11
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
housekeeping: Improve docs and make help #228
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
firebase-deployment: https://evy-lang--228-4m6p9b1r.web.app (0b18c9c) |
juliaogris
force-pushed
the
housekeeping
branch
2 times, most recently
from
December 14, 2023 05:33
4dee724
to
8fde1d1
Compare
camh-
approved these changes
Dec 16, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🍏 with a couple of minor comments
Update `make help` scripting so that target comments are listed above the target, rather than to the right of it. This makes it possible to write longer comments and make them easier to read in the Makefile source code. This update has been taken Camh's offscreen project. Link: https://github.com/foxygoat/offscreen/blob/12db5ebcf5da8d4cf1fc66f82965b849cf86e90f/Makefile#L108 Issue: evylang/todo#2
Replace sponsorship heart emoji as it shows up read at least on Chrome/Android is nearly invisible on the red pink background. Issue: evylang/todo#3
Make first README.md link point to evy.dev rather than scratch. We want to encourage people to play with Evy with the first clickable link, not read up on scratch. Issue: evylang/todo#5
Add releasing documentation, describing the releasing process and the meaning of minor and patch versions in Evy. Issue: evylang/todo#8
Add version endpoint to frontend so that it can always be determined which version of evy is currently in use via https://eyv.dev/version Issue: evylang/todo#7
juliaogris
force-pushed
the
housekeeping
branch
from
December 17, 2023 04:24
8fde1d1
to
0b18c9c
Compare
camh-
approved these changes
Dec 17, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🍏
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Update
make help
awk scripts for multi-line comments on top or rather thannext to target.
Add
/version
endpoint to deployment.Fixes evylang/todo#2
Fixes evylang/todo#3
Fixes evylang/todo#5
Fixes evylang/todo#7
Fixes evylang/todo#8