Skip to content

Document agreed Java guidelines to README.md #221

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

Closed
2 tasks done
npathai opened this issue Aug 19, 2015 · 5 comments
Closed
2 tasks done

Document agreed Java guidelines to README.md #221

npathai opened this issue Aug 19, 2015 · 5 comments

Comments

@npathai
Copy link
Contributor

npathai commented Aug 19, 2015

To keep the things in sync and properly formatted we need to follow some style guide.
My vote is for Google Style Guide as it is used in many open source projects. Other option is to follow Oracle coding convention.

Google Style guide configuration for IDE

Tasks to complete

  • Add agreed guideline in How to contribute section
  • Provide links to IntelliJ and Eclipse configurations for the same
@iluwatar
Copy link
Owner

I can live with the Google guide.

@iluwatar
Copy link
Owner

Maybe we can change this issue title to document the agreed conventions to README.md

@npathai npathai changed the title Follow some Java style guideline Document agreed Java guidelines to README.md Aug 20, 2015
@npathai
Copy link
Contributor Author

npathai commented Aug 20, 2015

@iluwatar I have changed the title. Should we target formatting existing pattern implementations with this issue?

npathai added a commit that referenced this issue Aug 20, 2015
Agreed guideline is conveyed and configuration link for eclipse and IntelliJ is also added.
@iluwatar
Copy link
Owner

I think we should. But this can be another task.

@npathai
Copy link
Contributor Author

npathai commented Aug 21, 2015

@iluwatar Sure. Maybe I will be able to squeeze in some time this weekend for it.

@npathai npathai closed this as completed Aug 21, 2015
pratigya0 pushed a commit to pratigya0/java-design-patterns that referenced this issue Aug 3, 2023
…uwatar#221)

* Edited couple answers in the Network section

* Edited Latency, Throughput and bandwidth

* Edited few more answers in the Linux section

* typos
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants