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 CI integration #2

Closed
zugaldia opened this issue Apr 5, 2017 · 1 comment · Fixed by #10
Closed

Add CI integration #2

zugaldia opened this issue Apr 5, 2017 · 1 comment · Fixed by #10
Assignees
Milestone

Comments

@zugaldia
Copy link
Member

zugaldia commented Apr 5, 2017

Once #1 gets merged, we need to add this project to Bitrise.

cc: @Guardiola31337 @tobrun

@zugaldia zugaldia modified the milestone: v0.1 Apr 5, 2017
@tobrun tobrun added the build label Apr 7, 2017
@tobrun tobrun self-assigned this Apr 10, 2017
@tobrun
Copy link
Member

tobrun commented Apr 10, 2017

Basic bitrise integration landed and is now active for every push to this repo.

To close this we need:

  • versioned bitrise.yml setup
  • checkstyle
  • unit testing
  • roboto tests on firebase
  • slack integration

@tobrun tobrun mentioned this issue Apr 10, 2017
cammace pushed a commit that referenced this issue Jul 14, 2017
# This is the 1st commit message:
add Building plugin

# The commit message #2 will be skipped:

#	fixed issues reviewed by me

# The commit message #3 will be skipped:

#	fixed bitrise script

# The commit message #4 will be skipped:

#	added DDS for building height

# The commit message #5 will be skipped:

#	removed commented out code
cammace pushed a commit that referenced this issue Jul 18, 2017
* # This is a combination of 5 commits.
# This is the 1st commit message:
add Building plugin

# The commit message #2 will be skipped:

#	fixed issues reviewed by me

# The commit message #3 will be skipped:

#	fixed bitrise script

# The commit message #4 will be skipped:

#	added DDS for building height

# The commit message #5 will be skipped:

#	removed commented out code

* cleaned up gradle file

* updated Makefile

* added optional constructor param to place buildings below layer

* added comment with more information
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 a pull request may close this issue.

2 participants