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 elementary Code #88

Merged
merged 3 commits into from
Apr 30, 2018
Merged

Conversation

Pittan
Copy link
Member

@Pittan Pittan commented Apr 21, 2018

Closes #86

@Pittan
Copy link
Member Author

Pittan commented Apr 21, 2018

@treyhunner Please review 🙇
@cassidyjames Is this all right? 🤔

@Pittan Pittan changed the title Add elementary code Add elementary Code Apr 21, 2018
@cassidyjames
Copy link

@Pittan works for us! Thank you so much!

Copy link

@cassidyjames cassidyjames left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to @elementary 😀

@Pittan
Copy link
Member Author

Pittan commented Apr 29, 2018

@xuhdev Please review and merge 🙇

Copy link
Member

@xuhdev xuhdev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very cool, thanks! Just a minor comment.

index.html Outdated
@@ -125,6 +125,7 @@ <h2>No Plugin Necessary</h2>
<ul class="editor-logos">
<li><a href="http://barebones.com/support/technotes/editorconfig.html"><img src="logos/bbedit.png" alt="BBEdit"><span>BBEdit</span></a></li>
<li><a href="https://github.com/eranif/codelite/tree/master/EditorConfigPlugin"><img src="logos/codelite.png" alt="CodeLite"><span>CodeLite</span></a></li>
<li><a href="https://github.com/elementary/code"><img src="logos/elementary-code.png" alt="elementary Code"><span>elementary Code</span></a></li>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does it have an individual page? I couldn't find it but it seems like there should be one. If not, can you change the link to https://github.com/elementary/code#readme?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cassidyjames Does it have an individual page?
@xuhdev I changed the link. If it doesn't have a page, please merge it 🙇

@cassidyjames
Copy link

@Pittan the editorconfig implementation is available at https://github.com/elementary/code/tree/master/plugins/editorconfig, but there's not a specific landing page for it or anything.

@xuhdev xuhdev merged commit d48c9c2 into editorconfig:master Apr 30, 2018
@xuhdev
Copy link
Member

xuhdev commented Apr 30, 2018

Cool, thanks! 🎆

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.

None yet

4 participants