-
Notifications
You must be signed in to change notification settings - Fork 7.1k
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
Add CONTRIBUTING + EditorConfig #278
Conversation
Thanks @wopian. Having a look. |
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.
I found the line breaks section a little bit confusing, but otherwise they're all good additions.
I also didn't know the AGC assembly now had syntax highlighting - someone must have been really bored! (not to mention that this repo is now somewhat of a meme)
That bored someone was me :) It wasn't much work actually since I based it
on regular assembly syntax highlighting.
|
@chrislgarry 👍❔ Can be added to later with more examples/things to do, but its better than having nothing. 👇 these 2 commits are the last. |
VSCode extension no longer requires manually adding settings
LGTM |
This adds a CONTRIBUTING file which will appear above every new pull request.
It provides some rough guidelines on what to check for in the code, including the outcome of #159 about line breaks.
Additionally it also adds a EditorConfig file to the root, so editing files directly on GitHub will enforce the recommended settings for AGC Assembly.