-
-
Notifications
You must be signed in to change notification settings - Fork 22
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 README.md #195
Update README.md #195
Conversation
✅ Deploy Preview for guidemaker-ember-template ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Can the current README template be used, but with the description of the old one?
Some sections no longer apply:
- The old compatibility info should be replaced with the new one
- The old running instructions are not applicable anymore. The new ones can be found here: https://github.com/ember-learn/guidemaker-ember-template/blob/main/CONTRIBUTING.md.
Hi i made the changes and added the commits to this PR. Not sure if you wanted to put the releases section back but I did anyways, please let me know if further changes are needed. Thanks |
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.
Left some minor remarks 😄
Co-authored-by: Ignace Maes <[email protected]>
Co-authored-by: Ignace Maes <[email protected]>
Co-authored-by: Ignace Maes <[email protected]>
@raspberri05 it looks like you deleted your fork so we're not able to merge this any more? 🤔 was that a mistake? |
@mansona i made a new PR :) |
issue #193. Added back the readme contents lost during v2 conversion. Please let me know if this if fine or if some things need to be removed. I currently just switched it back to the readme that had all the information.