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

Add CONTRIBUTING + EditorConfig #278

Merged
merged 9 commits into from
Mar 6, 2017
Merged

Add CONTRIBUTING + EditorConfig #278

merged 9 commits into from
Mar 6, 2017

Conversation

wopian
Copy link
Collaborator

@wopian wopian commented Feb 14, 2017

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.

@wopian wopian added Status: Review Needed Type: Meta Translations and other changes outside source code labels Feb 14, 2017
@wopian wopian requested a review from chrislgarry February 14, 2017 00:19
@chrislgarry
Copy link
Owner

Thanks @wopian. Having a look.

Copy link
Collaborator

@oldmud0 oldmud0 left a 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)

@chrislgarry
Copy link
Owner

chrislgarry commented Feb 16, 2017 via email

@wopian
Copy link
Collaborator Author

wopian commented Mar 5, 2017

@chrislgarry 👍❔

Can be added to later with more examples/things to do, but its better than having nothing.

👇 these 2 commits are the last.

wopian added 2 commits March 5, 2017 18:26
VSCode extension no longer requires manually adding settings
@chrislgarry
Copy link
Owner

LGTM

@chrislgarry chrislgarry merged commit e0d32c3 into master Mar 6, 2017
@wopian wopian deleted the contrib branch March 6, 2017 01:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Meta Translations and other changes outside source code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants