Skip to content
This repository has been archived by the owner on Aug 10, 2022. It is now read-only.

Solucionados los issues #11 & #14 & sonar análisis. #1

Merged
merged 6 commits into from
May 30, 2017

Conversation

joaquinolivera
Copy link

Se solucionaron issues, archivo de configuracion para el analisis de sonar y actualizado el .gitignore para los archivos .idea y .apk

@@ -35,6 +34,11 @@
* Final class: to prevent someone from accidentally instantiating the class, we give it an
* empty constructor.
*/
public void ipfromsystem(){
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member

@alefq alefq left a comment

Choose a reason for hiding this comment

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

Commit comments should include a reason, should tell why not describe what was changed. To see the what I have the file tracking provided by git.

@@ -0,0 +1,5 @@
projectKey=bip4cast
Copy link
Member

Choose a reason for hiding this comment

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

.sonar directory should be ignored from versioning, for same reasons we removed .idea and binaries.

@alefq alefq merged commit 68d64de into jokoframework:master May 30, 2017
alefq pushed a commit that referenced this pull request Jun 14, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants