Skip to content
This repository has been archived by the owner on Sep 23, 2021. It is now read-only.

Update workflow continuous integration #53

Merged
merged 21 commits into from
Sep 21, 2018
Merged

Conversation

hectorerb
Copy link
Contributor

@hectorerb hectorerb commented Sep 17, 2018

Changes description

  • Use references to handle jobs
  • Generate docs, coverage and test report independently
  • Update fastlane dependency
  • Update CircleCI workflow

Checklist

Please check if your PR fulfills the following specifications:

  • Tests for the changes have been added
  • Docs have been added/updated

Estimated time

Assignee 🍅
@hectorerb 32

References

Closes #N/A
Related #52
Depends on #N/A

@hectorerb hectorerb added the ci label Sep 17, 2018
@hectorerb hectorerb added this to the 1.0 milestone Sep 17, 2018
@hectorerb hectorerb self-assigned this Sep 17, 2018
ajsb85
ajsb85 previously approved these changes Sep 17, 2018
@hectorerb hectorerb force-pushed the feature/ci-section branch 7 times, most recently from 4344a9c to 8872388 Compare September 21, 2018 19:58
@hectorerb hectorerb changed the title WIP Update workflow continuous integration Update workflow continuous integration Sep 21, 2018

# LICENSE
#
# screenshots.sh is part of FlyveMDMAdminDashboard

Choose a reason for hiding this comment

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

Review the new @Naylin15 format for headers, now the name of the file is not present to be more flexible and readable

rafaelje
rafaelje previously approved these changes Sep 21, 2018
@rafaelje rafaelje requested a review from Naylin15 September 21, 2018 20:18
@rafaelje
Copy link

@Naylin15 please review the documentation script

@Naylin15
Copy link
Contributor

Sure 👍

@Naylin15
Copy link
Contributor

Hi @hectorerb

Here is the format for the .sh license headers Hackmd

@hectorerb
Copy link
Contributor Author

Hi @Naylin15
Thanks, license updated

@rafaelje rafaelje merged commit 8631361 into develop Sep 21, 2018
@rafaelje rafaelje deleted the feature/ci-section branch September 21, 2018 21:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants